NPM is a whole different beast.  There are ways to use it with rails but
let's not get into that now.
1.  If its easier for you to do yourself, thats fine.
2.  Use [rails-assets](https://rails-assets.org/) like we did for other JS
libraries
3. The manual way to #2 is to download the the libarary into your
vendor/assets/javascripts directory, then require it in your application.js
manifest.

On Thu, Jul 9, 2015 at 10:41 PM Brendan McGivern <notifications@github.com>
wrote:

> I tried importing the library with the browser globals method but I kept
> getting an error. I will try installing with npm. I fixed the issue though: className={"alert"
> + " " + this.props.bsStyle + " " + "alert-dismissible"}
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brendanmcgivern/blocitoff/commit/4ca0d7e6f6983ca7e9aa3de94e9b6f5c9273b0b0#commitcomment-12097700>
> .
>


です！
試しに↓のコードを追加した場合には期待通りの動作をしました。

```coffeescript
else
  @editor =null
```


Interesting -- but that seems too magical to include it, you're right.

BTW, I was just thinking about the compatibility of the files. The binary
files should be fine, i.e. you should be able to download the system build
from kaldi-asr.org and start using it in windows -- my question is about
the text files -- these won't probably be "portable", right? Because of the
different line ending.
But I think it shouldn't matter much -- the text files are usually used
only for debug, AFAIK. And, unless things changed, I believe Mac has
different line ending as well and we didn't have any complaint about that.

y.

On Thu, Jul 9, 2015 at 10:40 PM, Kirill Katsnelson <notifications@github.com
> wrote:

> I have a powerful linux box, where I do most of actual kaldi
> experimentation. Indeed, I'll run pre-commit tests.
>
> BTW, by any chance is there a way how to make the ios::binary flag default
> on win?
>
> Probably not in a compiler-independent way. Here's a leftover from my
> initial porting efforts (great minds think alike, uh? :-) ):
>
> A library conains just one file:
>
> #include <fcntl.h>
> #include <io.h>
> #include <stdio.h>
> static int initbinstdio(void) {
>   _setmode(_fileno(stdin), _O_BINARY);
>   _setmode(_fileno(stdout), _O_BINARY);
>   return 0;
> }
>
> #pragma section(".CRT$XIU", long, read)__declspec(allocate(".CRT$XIU"))
> int (*binmode_stdio)(void) = initbinstdio;
>
> .CRT$XIU is one of "magic" section names; these sections contain pointers
> to initialization function, and these functions are called quite early by
> crt0. Any binary compiled with this library and force-referencing the
> _binmode_stdio will have its stdin and stdout open in binary mode. Not
> sure if it will set cin and cout to binary mode, but I have a hunch it
> will.
>
> I thought about compiling OpenFST with this trick, as the fst* programs
> are often used in scripts with pipes.
>
> I do not know if the Intel C compiler honors these special sections. It is
> also likely, as they went quite far in making their compiler pretty much a
> drop-in replacement for the Microsoft one.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/5d2cc33b614294c36aa6e7e0e5619d69e9ca7aaf#commitcomment-12097695>
> .
>


Already merged


Вот  об этом я и говорил ))


I'm not sure this difference changes behavior, but `syou_down_num()` clears `ba[x][y]=0` here.


なるほど。ありがとうございます。
勉強になります。
この辺、コードを見てて気づくの難しいですね。多くのパッケージがリソース開放という意味では、曖昧にやっている気がする。。@aki77 さんはプロですね。


Consider using the servers' iterator instead. Might be interested in `position` method on `Iterator` trait.


This check is only used in this function. Why broaden the scope?


This function isn't actually essential for the drag-and-drop process, it's just used to tell the source process to change the cursor or drag image to indicate whether the drag will be accepted, depending on the current position of the cursor. For example, weston-dnd (which lets you drag flowers around) sets the drag image to be a translucent flower by default, but once it receives the accept message it turns into an opaque flower so that the user knows it's a good place to drop the flower. If it receives a reject message, it changes the flower to be translucent again.

Since there will indeed be a bit of a delay (though nothing really noticeable) due to IPC, this could result in the coordinates that are treated as accepted being off a bit. For example, the compositor sends the DragUpdated event to the GPU process giving us new coordinates for the cursor, then the GPU process informs the browser process, then the browser process checks the coordinates and calls either `DesktopDragDropClientWayland::IndicateDragWillBeAccepted` or `DesktopDragDropClientWayland::IndicateDragWillBeRejected`, sending an accept/reject message to the GPU process. Before the GPU process receives the accept or reject message from the browser process, it could receive another DragUpdate event from the compositor, changing the coordinates. Then the GPU process receives the accept or reject event from the browser process based on the stale coordinates, and forwards it to the compositor. I don't see any way to fix this, and it's hardly a serious problem (I'm not sure if anyone will ever notice the difference, even with a very slow computer), so I think it's fine. (Well, one possible "fix" would be for the browser process to send to the GPU process the coordinates for which to accept or reject the drag, and then the GPU process can discard them if they are stale. But this could be worse than the current behavior, since it could prevent accept/reject messages from ever being sent even if they are just a single pixel off, so I don't think it's a good idea.)

There will also be delay at the start of the drag, since we will never send an accept message until all the data has been transferred. This is because we have to pass the data to the `DragDropDelegate::OnDragEnter` before calling `DragDropDelegate::OnDragUpdated` to determine whether the data will be accepted or not; it would be quite a task to change this. I think this is acceptable, too, since the only downside is that if a process transfers a huge amount of data, the cursor might not change soon enough.


There seem to be more than fixing #23?


啦啦啦啦


还不错


Why don't you like the piping? :( Also, the reason we're using the perl code is that we based it off a perl tool that MathJax uses to generate its fonts (which we also use). If there was some automatic conversion then that sounds fine (we could probably leave the stuff we don't use commented out).


Is this correct?  While we don't have a proper schema, every one I've seen meets the 15 char letter and number rule.


PS: Mazen, you're stupid.


++ Or we could use `python -m json.tool` or `JSON.stringify(data, false, 4)` or something.


Should the unit test be so specific to the exact xml? Or should it just check when the data in the rows are correct or not? If that is too hard to test, then this is fine.


Leave the columns as 2 and 10 so they line up with the other panels


You may find this relevant:
http://www.codefellows.org/blog/5-ways-to-manage-front-end-assets-in-rails

There are lots of other ways you can manage your JS assets too.
Unfortunately I can't point you to a single / definite best practice.. but
the rails-assets site is simple and I like that you can keep up to date
using the Gemfile.

On Thu, Jul 9, 2015 at 10:47 PM David Roberts <drob@drob.net> wrote:

> NPM is a whole different beast.  There are ways to use it with rails but
> let's not get into that now.
> 1.  If its easier for you to do yourself, thats fine.
> 2.  Use [rails-assets](https://rails-assets.org/) like we did for other
> JS libraries
> 3. The manual way to #2 is to download the the libarary into your
> vendor/assets/javascripts directory, then require it in your application.js
> manifest.
>
> On Thu, Jul 9, 2015 at 10:41 PM Brendan McGivern <notifications@github.com>
> wrote:
>
>> I tried importing the library with the browser globals method but I kept
>> getting an error. I will try installing with npm. I fixed the issue though: className={"alert"
>> + " " + this.props.bsStyle + " " + "alert-dismissible"}
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/brendanmcgivern/blocitoff/commit/4ca0d7e6f6983ca7e9aa3de94e9b6f5c9273b0b0#commitcomment-12097700>
>> .
>>
>


Format this text too


Dixi quod…

>OK. In that case, I’d suggest we adopt drive letter notation
>and ; as path separator in mksh but keep / as directory separator.

I merged that now, as well as many other patches from you.
I’ve not yet touched anything that needs os2.c (well written,
by the way, thanks!), that will come when I have time to
actually invest into new ports (EBCDIC too), which is probably
after the mksh R52 (not R51!) release. Just so you know.

But this way, we can already reduce diff between mksh and mksh-os2.

Have a nice weekend,
//mirabilos
--
„Cool, /usr/share/doc/mksh/examples/uhr.gz ist ja ein Grund,
mksh auf jedem System zu installieren.“
	-- XTaran auf der OpenRheinRuhr, ganz begeistert
(EN: “[…]uhr.gz is a reason to install mksh on every system.”)


hey @F3PiX, I was going to contact you about this :) I put your name as a placeholder for the credits, but I'm happy to change the name to whoever should be credited for the picture. Or, yes, you can do it yourself if you want! :)


forcefixed. Thanks.


Please check & fix  the path in /config/*.mk and minimal_device.mk files.


Merge these into one `private`.


Thanks yo!! :heart: :heart: :heart:


Please remove "src/python/vsprojects/kos/dictionary" file.


I Just go in the file, like I did in the draft?


yes, exactly. ping me if you have any problems :)


are you planning on permanently keeping a `travis` branch? or is this a typo (cause it contradicts the commi message takling about master)


I am at vlatska ;)

T

Sent from Yahoo Mail on Android

From:"Tomas Rehak" <notifications@github.com>
Date:Fri, Jul 10, 2015 at 9:50 PM
Subject:Re: [chess] fixed another castling moved - wrong assesment of endangered fields from pawn (4ee6b2e)

Check whether enemy pawn does not endanger fields that must not be endangered in order for castling to be valid.

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


it didn't crashed and the loop finished, so I tougth it was ( I get the pointer from a ptr_vector ).


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 130](https://teamcity.labkey.org/viewLog.html?buildId=260263&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 1 Build time: 00:03:05
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest.initializationError: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

```


wouldn't you rather pull this out into a separate upload script which would only run on Travis (if the intent is to avoid users running this script attempting and failing an upload)?


4973_heatpump_defrost_bug (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (650 of 650 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-5.34%25-green.svg)


That's a hard one.
For the new stack trace of the dot product, I think I would rather copy the stack trace of where the initial dot product was created, and maybe add the one of the initial transpose.
For the stack trace of `y.T` and `x.T`, I think it is OK not to put anything.


Thanks - I'm updating it right away.


done


As for changing the API, I think adding a bit more functionality to the collection such that you don't need to know the internal structure of it (getDataMerged() handles that a bit).  Also splitting out the data operations from the visualization options makes a lot of sense.


Change that to "Community"


hhhhhh


Done! Thanks.


all {} were done in one go using IntelliJ


perhaps cache this?


I think that should be `node.outputs[0]` instead.


Actually, `firebase init` (possibly followed by `firebase deploy`)


The whitespace seems to be coming from here.


got it. Thanks @JsonFreeman


Please next time do it with the commit --amend, it will save GIT a lot of data, if it can describe the update files via a delta...


Instead of appending the nested component as a child node, we were just pasting its HTML into the parent component, which affected how the event bubbling worked.


I think we might want to error if we're not getting back a valid message


What's the difference between "save" and "saving"?


Haha, no worries! Good you learnt something from it :smiley:


YES. @sornaks can you send another PR to change this? Arrays are fairly cheap.


Привет. Всё хорошо, змечаний нет.

// я честно не понимаю формулу к этой задачи, там точно нету ошибки в самой формуле в дз????

Нет, там точно нет ошибок


done


this switch is in here for transforms like the following:

```
doc['data'] = _.omit(doc['data'], ['name']); return doc
```


He-he, this is the smell of v4 :)


Epic. :+1:


Nice! lambda is one of things in python you don't use often but it's good to know about.


I think I would just keep the trace of `x`, or even the trace of `node.outputs[0]` in that case:
- `x` would make sense because it is a `make_vector`, like `ret`.
- `node.outputs[0]` would make sense because we are replacing it with `ret`.


I fixed this now, and squashed the two commits into one


To keep image size to a minimum and due to Docker imges layered filesystem, it would be great to insert a git command that trims back the history to 1 after running the 'make' command. That's should be OK if the git commit ids aren't never needed again after building.
However, I don't know if such a command exists.


Looking at the previous commit, looks like "save" is the default, nothing is happening state. I know you didn't introduce this in this PR, but something like "ready" would be less likely to cause confusion.


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


I would put `node.outputs[0]` here instead.


Same here.


Looks great, way to use all the tools available to you.


right,
the default case looks like it's accepting a map[string]interface{}, and just setting the msg.Data to that value
I was wondering whether we should ensure that the return from the function is a valid msg, and if it isn't, then we should error..


`protected` isn't necessary for subclasses to be able to use a method; subclasses can use their parents' private methods. `protected` means that a method can be called by *other* objects of the same type as the class where the method is defined.


I think the `is not` syntax is prefer to `!=` for None but this works


В каждой таблице обязательно наличие первичного ключа - поля "id", либо составного первичного ключа.


I work at Interactive.


This breaks formats. For example, past gen formats allow future gen stuff.


Just curious, why did this have to be added?


lol i just finish my first version and it work so far (not pretty but it work)


straight to production


I don't think API-support alone for "min" and "max" rating will solve the problem of people still using game titles for it.

Seems more like a UI that trains the user and disallows `<`, `>`, `+` and `-` in game titles (Unless it's parseable?) would be needed. Of course, I don't know what you client does in this regard -- what's the current behavior?


Picked it up in my Ruby days. May have been from Avdi Grimm.


Всё вроде хорошо, молодец
Только вот эта строка меня смутила:
if (F1!=N) System.out.println(F2==0);


Readme file


None of the calls to `alloc()`, `cast()` and `node.op()` preserve the original stack trace.
I would suggest the following:
- for `v`, maybe the trace of `node.outputs[0]`, as it is where `node.op()` had been called initially
- for `T.cast(...)`, either the same as `v` or nothing
- for `ret`, keep `node.outputs[0]`.


I did this because it was making the diff against the previous release harder to understand; I prefer to separate functional changes from stylistic ones. I've now [standardised the words](https://github.com/faye/websocket-driver-ruby/commit/7128d1b0307954601c7a42051c9c3c8eacaff08f) in these methods. I've gone with:

* `chunk`: some binary blob fed in from TCP, whose boundaries are totally arbitrary
* `buffer`: a binary blob we've constructed ourselves, that represents some specific thing like the next chunk of frame headers, or a frame we're building
* `octet`: a single byte. I chose this because I also have to maintain the Node.js version of this library and `byte` is a JS reserved word


performance test results:

![image 4](https://cloud.githubusercontent.com/assets/1051390/8628266/81857312-271f-11e5-8d22-5a1c2d274786.png)


Got it.  Thanks.


Ah, I see. At first, I saw it in the abstract method signature of handle() and thought that it must've been forgotten in the method definitions, but I was also thinking of it from a slightly different perspective. The purpose of statscache is to aggregate fedmsg traffic over some intervals (1s, 5s, 1m, &c.) and then to deliver copies of the traffic during each interval to the plugins for processing. My interpretation was that plugins might then be written under the assumption that the messages passed to ``handle()`` are all from the most recent interval, from ``self.frequency.last().last()`` to ``self.frequency.last()``. When statscache is brought up and initializes its plugins, each currently queries datagrepper for the fedmsg traffic missed while offline, which it then processes retroactively. Since the plugins need to check the timestamp on each individual message, like you said, this works fine, and all the old data can be processed all in one go. But if a plugin *were* written with the assumption that the messages given to ``handle()`` are all from the same interval, then processing the old data all at once will cause the plugin to generate incorrectly, since it would believe that all those message arrived in the last interval.

On IRC yesterday, I was talking with @ralphbean, and we were considering about an alternative method. This would be to have the plugins trust that all the messages passed to ``handle()`` are from the interval ending at ``timestamp`` and to have the statscache producers query datagrepper on startup and retroactively process the data, using timestamps of the past intervals. Since every plugin needs to do this retroactive processing, it seems like it would better isolate them from such implementation specifics while also reducing the amount of data needing to be queried from datagrepper.

However, based on your comment, this may actually be a non-issue. Thoughts? (sorry for the long comment, btw)


I don't know why I had issues to build on Ubuntu 15.04 but I had to add it.


done


instead of passing in `None` to represent `Self`, just take an `ast::Name` and pass in `syntax::parse::token::special_idents::type_self.name`


We are creating a new node by calling `assert_()`, so we should copy the trace from `node.outputs[0]` (the original assert).


So I reworked exception handling a little (350386f5).  It still catches anything thrown, but it now forces a crash in the main thread with the exception in the DynEMC thread set as the cause.  I also moved the try-catch block from ```EnergyValueRegistry.init()``` to ```DynamicEnergyValueInitThread.run()``` in order to catch errors like #948.


[#7] fix colorbox buttons don't show up


can you just call this "is_static", since there is no superregion of 'static


Oh, shit happened. I am sorry, I am probably tired now. I gave -ammend
option instead of --amend. Unfortunately this was syntactically correct. :(

2015-07-10 22:08 GMT+02:00 mfonville <notifications@github.com>:

> Please next time do it with the commit --amend, it will save GIT a lot of
> data, if it can describe the update files via a delta...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengapps/arm64/commit/f3cc668208f6c737cf7cbe4a212aef24223acb72#commitcomment-12113245>
> .
>


I would amend this to say "...and hence which can only be 'static"


We are calling `i.owner.op()`, so I think we should copy the stack trace from `i`, actually, since `new_i` replaces `i`.


Perhaps you meant this

```javascript
 if (request.headers.hasOwnProperty(key) && ignoredHeaders.indexOf(key) < 0) {
```


yes! done. thanks.


Sorry, it was for MSVC 2013, for snprintf.


can you add a sanity check test like:

```rust
fn static_id<'a>(t: &'a ()) -> &'static ()
    where 'static: 'a { t } //~ ERROR
```

this is just protecting against us getting sub/sup mixed up somewhere :)


Anyone going to push this updated version to supermarket any time soon? :)


I broke down the code into easier to read pieces here 93f9935ab535bb6730229f64905c85979ca58259 does that make it clearer or should I rework it?


No, all new development is happening over at https://github.com/poise/poise-python for now. When that is a little more completed, I might try to make this into a backwards compat wrapper.


i realy should check before i update somme time.....


A cmake nem hozza létre a preferences fájlt, a hagyományos mód igen.
A preferences fájlt sed -el kell megszerkeszteni, hogy az ikon útvonalak éljenek.
Ráadásul a genpref binárist már semmi nem installálja, kézzel kell kopírozni. Ez hozza egyébként létre az említett preferences fájlt, melyet úgy tűnik, hogy a fejlesztők dobtak a konfig fájlok közül, de mégis szükséges valami.
A magyar po fájl nem tartalmazza az fdomenu.cc magyar menübejegyzéseit. Ezt esetleg lehetne pull-request -el tetni az eredeti forrásba.

Az uhunmenu a pythonos generátorral dolgozik az lxmenu-data alapján.


Seems like your Checking deposit/withdraw methods are working great. Shouldn't have a problem implementing this for the Savings account.


I would copy the stack trace from `node.outputs` actually. There can be more than one output, so it would be something like:
```python
    for out_idx, out in enumerate(ret):
        copy_stack_trace(node.outputs[out_idx], out)
```


This commit also reformats the config file (chtcrequirements.txt) to <value>,<display value>


didn't mean to commit that


Forgot to mention that I also made write_all in a thread since nil was concerned it may block if the server list is too big.


done


I now see btw, that this apk is also missing (did not notice it before)


Set an environment variable instead. OSAD expects a particular one documented [here](https://github.com/stackforge/os-ansible-deployment/blob/f5388b61e4c41c071f3935066b1150f1a6cffe01/scripts/run-upgrade.sh#L36-L42).


coding style


no spaces


Curious. Type of `return` wasn't getting resolved to concrete `IO`?


Either provide descriptions either remove this jadadoc lines


Hello Xiaolu.
I tried to build this repository against cm-12.1 and while it builds, I cannot boot it on m9pw.
Can you give me some hints how do you exactly build this to boot on m9pw? thank you!


93469234-DemandResponse2 (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1693 of 1693 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)


That's the actual fix :-)


no space, redundant spaces


Fix casing


@PEMapModder

How can I make it so users can set port for each server listed under "servers"? Rather then 1 port option. (Not all servers have a 19132 port)


@jcoglan awesome, thanks so much.  Any plans to do an early gem release for this?


Unnecessary comment


don't call variables with underscore


Credits to TheInfamousPig for letting me nick his buttons


I use a latin1 encoding terminal and this doesn't work.


nice!


I have not (yet) read all the updating mechanism, so I might have misunderstood, but updating from 0.8x directly to the latest /dev will probably not work, hum?


Is this needed for this PR?. Why you removed it? is web-console not working with Rails API?.


done


AFAIK Throwing runtime exceptions is bad practise in java.


In any case I'd discuss in a separate PR


not a big one here


i think it's better to just build it in it's own repo otherwise the OF builds will take forever. we could detect problems earlier if we built it every time OF changes but the same goes for examples... as long as we have the binaries with the latest changes in them i think we are ok, the changes that might break it are probably not so critical and the things that might be important to catch are not so related to compiling it but changes that might break it from working even if it compiles properly


this is mostly so the script doesn't upload the binaries when run on a PR i could put it on a different script but it's so small that i think it's ok


no space after for, as usual random space putting without any coding style


I'm looking at this now and is there a reason to hardcode the version number? This means someone has to be on top of making sure this doc is up to date.


use try with resources


Just wanted to add "Logging usage sample" to README.md, it`s my first commit.


4973_heatpump_defrost_bug (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1682 of 1682 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)


also, i'm using travis by now because i'm testing there once i merge it into master i'll change it


1712-AvailabilityManagers (kbenne) - 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-NA-red.svg)


JsDoc


never use space separated input values


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


get in


removed


does it have corresponding UI binding?


Better to use `z` instead of `$` here


It would be nice to move all the logic to a separate file.
dspio.c is actually just a mediator between the sb16 logic,
DMA, midi and dosemu's PCM interfaces. What makes it
even worse is that the intelligent mode doesn't even belong
to sb16 at all, while DSP is a part of sb16.


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


`css/bootstrap.css` was a custom build. So I reverted the last commit, shaving off about 40-50kb of the CSS file.


1712-AvailabilityManagers (kbenne) - Win64-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-NA-red.svg)


LGTM.


Based on my numbers it looks like you're missing something in the minification. Even with your numbers the difference is only ~3kb when you add the fact that your script now needs es5-shim.


Was reviewing the various forks off of EP4 and saw this one.  Not sure I understand the setup/need of this, but if I may make one or more suggestions:

Where you have HTTP_SERVER, replace with:
`($request_type == 'SSL' ?  HTTPS_SERVER : HTTP_SERVER)`
or further replace HTTP_SERVER.DIR_WS_CATALOG with:
`($request_type == 'SSL' ?  HTTPS_SERVER . DIR_WS_HTTPS_CATALOG : HTTP_SERVER . DIR_WS_CATALOG)`

Or more so as originally provided/intended replace HTTP_SERVER.DIR_WS_CATALOG with:
`($request_type == 'SSL' ?  DIR_WS_HTTPS_CATALOG : DIR_WS_CATALOG)`

Though the standard these days is if I understand what is "happening" above, to use the same catalog designation for both secure and non-secure and modify the applicable HTTP(S)_SERVER constant to reflect the difference.  This may be why the above "fix" was applied because of a setup such as where it is a shared account requiring a different path for secure files than the standard HTTP_SERVER designation.  Just for information though the strict use of HTTP_SERVER (the inserted code) causes the files to be transferred without the SSL applied upon download, regardless of where the files are stored.  May be acceptable for some, but not all and if a SSL is available/used is not a suggested practice to use HTTP_SERVER in this case.

Was this issue identified in the forum?  I recall a discussion about something like this, but it seemed to never get closed out.


Addresses https://github.com/OpenTreeOfLife/opentree/issues/670


I think its better to convert it to x-x-schema, given https://html5sec.org/#78


clarified and renamed


Well this was proof of concept. Now let's get it right.


Move the `ai_authUser` to a static field like the cookie separator since this string is used at multiple places.


Use GetOrAdd (https://msdn.microsoft.com/en-us/library/ee378677(v=vs.110).aspx) for the ConcurrentDictionary implementation.


done


Maybe rename `kvps` to something more descriptive?


Why is this needed?


Good point!  I can change that back if you wish.


@ryanstout I'll try to get it out as soon as possible, but releasing these libraries involves an overnight test suite so I can't do it immediately. Hopefully over the weekend.


P.S: Screenshots work but need updating. I don't recommend in using it yet because if you to, you'll need to restart your client.


Sorry to be so helpless but I cant delete it?


Sure! I reverted the changes regarding re-ordering and kept strictly Swift related changes.


It's not the 0.8 version but a dev version. It will not work but those who are using /dev branch should know they have to be careful.


fyi
id = models.UUIDField(primary_key=True,auto_created=True,editable=True)


@jcoglan cool, thanks!


ec42cab5196065c16f5888cad073b032c5a9817c LGTM.


5d3f1d8868ce872c5b2cc67d58fdc4fc9b7e85db LGTM


all looks good abovve, any issues?


Nice. Just a couple of things:

1. I think the tests should go into gtrackcore.test.track_operation so that they are together with the other unittests.

2. Another tip might be to refactor so that one can call the _run_union_test (which should really be camelCase...) method with the starts, ends, etc directly as arguments. In that way each test will just take one line (or broken down to a couple, if the line is long). You can then easily add readable tests for all bordercases. E.g. _runUnionTest(expStarts=[2, 5], expEnds=[4, 8], startsA=[2], endsA[4], startsB=[5], endsB=[8]). You might even refactor so that all operations uses the same _runOperationTest() method, simplifying creating new tests even more..


This isn't mine! When rebasing, remember to change authorship.


sorry :(.


Also this redundant, because any provisioning error is wrapped and printed in provisioning block. If you need to know what cloud fails, then it root trace must add cloud name.


nope


0dfd1a982e43d01c2f0321b1010f46b20a121fee LGTM


Note: that's a security update. http://openssl.org/news/secadv_20150709.txt


![screen shot 2015-07-10 at 4 48 18 pm](https://cloud.githubusercontent.com/assets/320464/8628755/94d0a406-2723-11e5-8309-d554a6bb4914.png)


The idea is that you can specify the rating range when creating a game
![image](https://cloud.githubusercontent.com/assets/1864183/8628681/4b7113f4-2755-11e5-96f2-5517e888ce67.png)

This is ignored however due to no server side support. For hosted games, I just parse the title:
![image](https://cloud.githubusercontent.com/assets/1864183/8628715/822f9e38-2755-11e5-953a-ce549a194c64.png)

If people would still type rating bounds in the game title, it would be ignored. Instead of disallowing those signs, we could detect such patterns and display a hint that he shouldn't do that and use the slider instead.


It's really nice, just wondering if it turns in the wrong direction?
And now I think the shutdown also needs something ...


I already fixed it.

Justin

> On Jul 10, 2015, at 2:49 PM, Bobbilee <notifications@github.com> wrote:
>
> topic_params
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/jgrant29/bloccit/commit/141d2099fbbb3a673d1ebd198ba893c0e26953e1#commitcomment-12114011>.
>


I submitted this upstream
https://review.openstack.org/#/c/200717/


c262e4536c706bbb938df72568d542cb8a209a1a LGTM


```
p, err := os.FindProcess(container.Pid)
 	if err != nil {
-		logrus.Errorf("Unable to RegisterMachine %s for %s: %s", container.Name[1:], container.ID, err)
+		return err
+	}
```
I don't think we need this.  This is an expected behaviour, if the container exited before RegisterMachine happened


Anne says we need to keep the $.variables.  Will put them in their own RS-specific patch


Yes, but wouldn't it be safer to update to 1.0.0 anyway, which has a chance of working?


15501000988072931c3cfd52bd132da973d67e78 LGTM.


I wasn't able to get this to build until I changed the `opencv3` dependency to `opencv`.


Didn't @rsandell suggest also to update the error to include the job name?


+	if err != nil {
+		pState, err := p.Wait()
+		if pState.Exited() {
+			exitedP := pState.Pid()
+			if container.Pid == exitedP {
+				return nil
+			}
+		} else {
+			logrus.Errorf("Unable to RegisterMachine %s for %s: %s", container.Name[1:], container.ID, err)
+			return err
+		}

There is a problem here also.  If +			if container.Pid != exitedP  you return nil, where you should probably still report the error.
Simply remove second else { block


You should rebase on the most recent version of Presto


Yes, it appears something has been murdering the files server lately.. We're working on it.


Ship it!


Thanks Jason!

Mike

________________________________
From: Jason Dunsmore <notifications@github.com>
Sent: Friday, July 10, 2015 3:50 PM
To: rackerlabs/docs-cloud-orchestration
Cc: Mike Asthalter
Subject: Re: [docs-cloud-orchestration] final updates for GA docs (249fb21)

Anne says we need to keep the $.variables. Will put them in their own RS-specific patch

-
Reply to this email directly or view it on GitHub<https://github.com/rackerlabs/docs-cloud-orchestration/commit/249fb21558314551f1e695929185bc6af5a7aeac#commitcomment-12114047>.


This is stack_adopt_resp.json upstream, no change necessary.


Just remembered that we probably want to call this from the default startup code.


Wdym


testing


1b1fb11c0eff8d470933d934ad62bdac6514c9af LGTM overall.

FWIW, the redundant doctest-snippet changes confirm my sense that those snippets don't belong in method docstrings.


It is used in two places: At the top level, to decide, whether a dataset must be migrated at all. In that case, the returned path is logged with an info message. And in the recursive call, to decide whether a subspec must be migrated. In that case it only chekcs for non-null before starting to process the subspec.


I don't know: I'm not sure people using dev branch want to switch to the stable version. Maybe we can consider to remove update from this old dev version and only keep the last ones (1.2-dev / 1.1.2-dev)


I'm not sure how I feel about what are essentially code comments in a separate file. They just sound like they are very liable to not get updated with the source code.


@psytronx DWMR. Well done here. Press on.


done


it's for the tests to easy initialize an employee


if the version is optional, it might be better to have:

```ocaml
let resolvable ~name ?version state =
... match version with None -> name | Some v -> name ^ "." ^ v ...
```


license


You removed the reading from app settings in the ReadFromConfigSection method which is fine, but you didn't call your ReadFromAppSettings method.


do what? I'd say "implement these services".


Oh wow. Took me a while to understand this change. It's nice! But we should find a more readable way at some point 😁


Minor cleanup yet causes a test failure.
Site Administrator can add example user as member of workspace        | FAIL |
http://jenkins.ploneintranet.net/job/Plone.Intranet.Pull.Request.Builder/390/consoleFull#1313619462d7588e9e-566a-4586-b220-5d422b09bef9


if you use ocaml-hex you don't need to strip newlines and spaces normally. See https://github.com/mirage/ocaml-git/blob/master/lib/SHA.ml#L135-138


I've just been working on giving a brief summary of the purpose of each
file in the source code. It's nothing too in depth, but we can talk it over
with itamblyn if you're not in favour of it

On Fri, Jul 10, 2015 at 4:54 PM, ludamad <notifications@github.com> wrote:

> I'm not sure how I feel about what are essentially code comments in a
> separate file. They just sound like they are very liable to not get updated
> with the source code.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hashkat/hashkat/commit/42dc7c83dfc17cbd7e939ca32181007d376fd3e9#commitcomment-12114116>
> .
>


"cacheSolve: This function computes the inverse of the special "matrix" returned by makeCacheMatrix above. If the inverse has already been calculated (and the matrix has not changed), then the cachesolve should retrieve the inverse from the cache." Notice the bit that says "(And the matrix has not changed)"


`Code.(` is not necessary with polymorphic variants.


Should we replace all the individual pw-*.dic lines with a generic line?


My problem is not with it being not worth it in the short term, If that's the only goal, that's fine. I just fear that it's burdensome to anyone making changes to then update this. It only takes a few changes for this to become more confusing than it is helpful (just thinking big picture).


A big thank you @tornoz  , I'll update it right now.  :)


4973_heatpump_defrost_bug (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1167 tests passed)

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


there is a nice library to route HTTP requests on top of cohttp (I haven't used it yet, but it looks nice): https://github.com/rgrinberg/opium


why is the annexcode constantly overwriting rccommand even if there is no update?


sounds good.


same here, you don't need the `Code.(`


@zachary-williamson Looks like you have the correct solution.  Almost certainly that was a bug!

Can you make it a separate patch on the default master branch?  Then you can get that patch through quickly without confounding it with these other changes.


Good idea.


This is a test repo


@vcunat I fixed it in 6befeb681810dbd13deda24330f40a3d741ef4e2, sorry for the noise.


these stubs I removed were redundant since they're already stubbed in the setup block.


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


@thedarkone No worries. It was a reasonable assumption to make at the time. And there's a lot of code out there doing it, so we'll need a way to cope with it.


Too late! :)

Let's tackle this after LA Roadshow.

On Fri, Jul 10, 2015 at 4:44 PM, Drew Williams <notifications@github.com>
wrote:

> Good point! I can change that back if you wish.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/intel-iot-roadshow/getting-started-guides/commit/545cc513b8ec64f2c1b62e47443d6466a0ad60a4#commitcomment-12113931>
> .
>


it is misspelled


Reverted. Had no issue locally but the build server was still experiencing the stackoverflow exception


this is going to re-render view every 200ms, right? can we make this adjustable?


Yes, either provide an update that has a chance of working, or no update at all.


> this is mostly so the script doesn't upload the binaries when run on a PR i could put it on a different script but it's so small that i think it's ok

you could move it into the after_success build phase so it only uploads when the build completes successfully, and easily check if you're in a PR build with `$TRAVIS_PULL_REQUEST`, see [here](http://stackoverflow.com/a/21941526/599884). I guess I just try to keep stuff modular and composable and separate unrelated tasks more clearly.


Thanks


This stack_name parameter is now on 6 different lines in https://github.com/openstack/api-site/blob/master/api-ref/src/wadls/orchestration-api/src/v1/orchestration-api.wadl  Mike, should I add the ```path="$.stack_name" type="string"``` part to all of them?


If it's not yet on files.minecraftforge.net, assume derped.


@livingincode DWMR. Very well done here. Please remove the static images (1.jpg, 2.jpg, ...) on the next commit. Press on!


No specific reason. It was either a mutable set or a mutable variable, since I didn't want to assign new variables on set removal.


> `coobook` with one 'k'

Is this a deliberate namespacing choice, or a typo that was just used [consistently](https://github.com/chef-cookbooks/chef-server/commit/1eef717eb8527092108234b079099b602a9c13a8#diff-55bf87238c9b8af164c5133a60721f12R21)?


@kuujo


@wardi Can I cherry-pick this for inclusion into master? Do we know if this commit a real issue or just a hack needed to get vocabulary loading working locally?


Added


Not in upstream, ignoring


>> There is a isdecu() which allows possitive and negative to be OK.

> I see the problem now. I should have named that isdecs().

I suggest we refactor it. I will very likely keep forgetting and forever assume isdecu() is for unsigned int.


Not a problem, I totally get what you are saying. I'll discuss it with
itamblyn

On Fri, Jul 10, 2015 at 5:02 PM, ludamad <notifications@github.com> wrote:

> My problem is not with it being not worth it in the short term, If that's
> the only goal, that's fine. I just fear that it's burdensome to anyone
> making changes to then update this. It only takes a few changes for this to
> become more confusing than it is helpful (just thinking big picture).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hashkat/hashkat/commit/42dc7c83dfc17cbd7e939ca32181007d376fd3e9#commitcomment-12114241>
> .
>


O Theme antigo mostra no fundo o papel de parede configurado no sistema


nit


Derp.

I've re-issued a new one.


No, no admin code change other than that one line in the modules.php file. I don't believer the currencies class is even being loaded in the admin for the modules.php. If there was a single class loader/manager for both the catalog and admin then that one line change won't event be necessary. PHP strict requires declaration to be compatible, which weren't the case for the catalog/currencies and admin/currencies. I also noted this in the install.txt.


Thanks. I definitely have, too (or accidentally running dd on my hard drive rather than my flash drive).


Ok set upa complete


Thanks! I have no idea how that happened


I think the stuff in the scripts dir is more recent but most of the scripts are just code for setting up the API calls. I fixed the command instead of trying to clean up the scripts.


@mhotar
I put some videos here:
http://downloads-21.open-tx.org/movies/
Do you think we can improve the shutdown? We have some unused flash :)


But yes, cherry-pick away, there's nothing wrong with this commit


That probably indicates a problem with the Gemfile or so - lets debug on Monday.


@leahpadgett DWMR. Simple stuff. Done well. :+1:


That's just for style so the code gets more concise. Also seems to be a general practice in Ember projects.


ok. I did a bit of cleaning up around the scripts, but won't spend much time on it


Cannot say I am opposed (I can see it being a useful/valuable feature) so much as making a note of a fringe case, should anyone else start looking.  Not able to do much debugging to know for sure what is going on, but I do know the code in question is largely hand written assembly and thus suspect the code doing something unconventional.
On Jul 10, 2015 3:59 PM, Michel Pollet <notifications@github.com> wrote:wierd. that RETI is generated by the toolchain...
But yeah perhaps they have something bizare. Perhaps the logic should be changed, or made optional. It's too bad it's very nice to be able to trace what the CPU is doing when in handlers and such...

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


for the scripts I just wanted to remove all the API stuff and just use ckanapi instead, maybe with env vars for where to connect/api key to use


We have to be careful with range-for on `ReadView::txs`, because the dereferencing of its iterator can throw an exception of the deserialization fails. This doesn't mean we should _never_ use range-for, but call sites have to decide if they want to abort the entire loop on an exception or if they just want to skip that transaction. Deserialization should never really fail since we check it before inserting into the map in the first place. But the interface allows it to happen. Call sites should be audited.


Just a random test from OctoDroid. Nice app.


This is already upstream


Added


`Ledger` used `std::shared_ptr<SHAMap>` for its maps which was pointless, since a `Ledger` always has a `SHAMap` upon construction. So `haveTxMap()` always returned `true`. A future change should do away with the `shared_ptr<SHAMap>` in `Ledger`.


Yes please!

Thanks Jason!

Mike

________________________________
From: Jason Dunsmore <notifications@github.com>
Sent: Friday, July 10, 2015 4:10 PM
To: rackerlabs/docs-cloud-orchestration
Cc: Mike Asthalter
Subject: Re: [docs-cloud-orchestration] final updates for GA docs (249fb21)

This stack_name parameter is now on 6 different lines in https://github.com/openstack/api-site/blob/master/api-ref/src/wadls/orchestration-api/src/v1/orchestration-api.wadl Mike, should I add the path="$.stack_name" type="string" part to all of them?

-
Reply to this email directly or view it on GitHub<https://github.com/rackerlabs/docs-cloud-orchestration/commit/249fb21558314551f1e695929185bc6af5a7aeac#commitcomment-12114341>.


Factored individual tests out into reusable methods. I reuse one set in one of my tests.


tested and working


@xxFlare Take a look at how TimeCommander's config file is set up and parsed.


Ah. I suppose having files would help =P

Got any ideas for how you wanna implement files?


xplat?


not working, but it's broken for normal users too. Selecting yes you want a response doesn't bring up the full form via javascript as it should, because the select value isn't what the javascript is looking for.


Lovely. I'll merge it on Monday. Thanks.


I had it as follows but the above seemed more readable
```
func newDataPoints(f *JSONWriteFormat, defaultDims map[string]string) []*datapoint.Datapoint {
	dps := make([]*datapoint.Datapoint, 0, len(f.Dsnames))
	for i := range f.Dsnames {
		if i < len(f.Dstypes) && i < len(f.Values) && f.Values[i] != nil {
			dps = append(dps, NewDatapoint(f, uint(i), defaultDims))
		}
	}
	return dps
}

func newEvent(f *JSONWriteFormat, defaultDims map[string]string) *event.Event {
	if f.Time != nil && f.Severity != nil && f.Message != nil {
		return NewEvent(f, defaultDims)
	}
	return nil
}

func (decoder *JSONDecoder) Read(ctx context.Context, req *http.Request) error {
	defaultDims := decoder.defaultDims(req)
	var d JSONWriteBody
	err := json.NewDecoder(req.Body).Decode(&d)
	if err != nil {
		return err
	}
	es := make([]*event.Event, 0, len(d)*2)
	dps := make([]*datapoint.Datapoint, 0, len(d)*2)
	for _, f := range d {
		e := newEvent(f, defaultDims)
		if e == nil {
			dps = append(dps, newDataPoints(f, defaultDims)...)
		} else {
			es = append(es, e)
		}
	}
	derr := decoder.SendTo.AddDatapoints(ctx, dps)
	eerr := decoder.SendTo.AddEvents(ctx, es)
	if derr != nil {
		return derr
	}
	return eerr
}
```


I agree, we should respect the value defined in the dictionary.

The second change can not be made because it will change the behavior of the opennlp models, it breaks compatibility with opennlp. The method is virtual, you can implement a subclass smoothly.
:+1:


OK, lets revert the direction then.
![unlock_0](https://cloud.githubusercontent.com/assets/6114100/8629177/5079da30-2759-11e5-8fc5-5c638fd07588.png)![unlock_1](https://cloud.githubusercontent.com/assets/6114100/8629180/531f1390-2759-11e5-8535-b212a7ed123d.png)![unlock_2](https://cloud.githubusercontent.com/assets/6114100/8629238/c378fbec-2759-11e5-802c-815e33550c66.png)![unlock_3](https://cloud.githubusercontent.com/assets/6114100/8629190/66027b1e-2759-11e5-94e3-aa905252367f.png)![unlock_4](https://cloud.githubusercontent.com/assets/6114100/8629194/6c61e79c-2759-11e5-8639-21f644580e2a.png)![unlock_zz](https://cloud.githubusercontent.com/assets/6114100/8629204/7417571a-2759-11e5-8d71-82a2b89b3372.png)

It looks good on the lcd, thanks for the videos.
My idea about functionality was a little different, I think the circle have to go a little faster, so user have to let it go to the end completely, then it will stay for some seconds to let the user time to release button. If the button is then not released , the ZZ image appears.

I will try to do something for the shutdown.


Double empty lines.


These handlers should hava JavaDocs.


Yay!


Hem... Surviv'Ant ? :D


JavaDocs.


Absolutely none. I've spent the past 4 days researching, and I can't find anything I can implement. I believe I need to make a VFS in the kernel, and add the file system as a module in grub, but I have no idea how to implement any of them.


Only runs on net45; skipped if Mono.


I'm running 0.40, and this seems not to work -- I get errors like so:

{code}
root@ init.d]# /usr/sbin/carbon-c-relay -f /etc/carbon/carbon-c-relay-default.conf -p 2015 -i 0.0.0.0
[2015-07-10 21:27:07] starting carbon-c-relay v0.40 (2015-06-26), pid=9736
configuration:
    relay hostname = my-test-host
    listen port = 2015
    listen interface = 0.0.0.0
    workers = 5
    send batch size = 2500
    server queue size = 25000
    statistics submission interval = 60s
    routes configuration = /etc/carbon/carbon-c-relay-default.conf

[2015-07-10 21:27:07] expected port, or unexpected data at '2004=a' for cluster local
[2015-07-10 21:27:07] failed to read configuration '/etc/carbon/carbon-c-relay-default.conf'{code}


Incomplete JavaDocs.


:clap:


The whole thing is now only use for Full .NET AnyCPU projects. NuGet 3 and DNX will handle all other cases.


Give examples.


Gotcha. Would it be possible to actually create a file on the host machine for the file system? For instance, have the OS generate a file called "FileSystem" or something, and read from there?


tryRun and tryCall


Remove


All the $.vars are in this downstream branch:
https://github.com/rackerlabs/api-site/tree/rs-heat-dollar-vars


You forgot to edit the docs.


Specified not passed.


Good catch.

On Fri, Jul 10, 2015 at 11:51 AM, strager <notifications@github.com> wrote:

> If the *dwFreeType* parameter is *MEM_RELEASE*, *dwSize* must be 0
> (zero). The function frees the entire region that is reserved in the
> initial allocation call to *VirtualAllocEx*.
> <https://msdn.microsoft.com/en-us/library/windows/desktop/aa366894%28v=vs.85%29.aspx>
>
> s/size/0/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sas/ds2/commit/990c45794ac9e546a5205327b91c3be6fcdb37cc#commitcomment-12111870>
> .
>

--
Stephane Sezer


Fixed here: https://github.com/yeoman/generator-node/commit/ff9ec788aaa044ac85e233c5a1cbb2f7c5bde815.


I believe i've addressed all the review comments


Don't say that.


There can be multiple presets (multiplicity 1-n) in the standard [1].  We have a way to put these into the display node [2] but currently the slicer4 Volumes GUI does not expose this (the slicer3 one did).

I don't think it is a big deal to only handle the first window center/width pair but maybe add some documentation noting that we do this on purpose.

[1] http://medical.nema.org/medical/dicom/current/output/html/part06.html

https://github.com/Slicer/Slicer/blob/3bfa2fc2b310d41c09b7a9e8f8f6c4f43d3bd1e2/Libs/MRML/Core/vtkMRMLScalarVolumeDisplayNode.h#L172


Mention NaN.


That's what qXfer:libraries:read is supposed to do. Just list the libraries without listing the main module.


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.11](http://pixelated.design/build/viewLog.html?buildId=27&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **FAILURE**
Summary: Tests failed: 6 (6 new), passed: 49 Build time: 00:00:09
### Failed tests
```
Lobster::Server::ThreadPool: #new: with parameters: with parameters spawns the specified number of threads: <no details avaliable>

Lobster::Server::ThreadPool: #thread_count: with no initial threads: with no initial threads increases when a task is added: <no details avaliable>

Lobster::Server::ThreadPool: #queue_length: #queue_length increases when all threads are busy: <no details avaliable>

Lobster::Server::ThreadPool: #queue: #queue passes arguments to the block: <no details avaliable>

Lobster::Server::ThreadPool: #queue: #queue spawns additional threads if the pool is busy: <no details avaliable>

Lobster::Server::ThreadPool: #join: #join causes new tasks to raise an error: <no details avaliable>

```


This was removed from upstream and replaced with a better description.


Hoping to get this out on Monday.


Thanks for cleaning that up. But shouldn't it be the other way around?

`LINK_WITH_MSVCRT` => depend on MSVCRxxx.dll => `/MD` or `/MDd`
`NOT LINK_WITH_MSVCRT` => link statically => `/MT` or `/MTd`


'hebrew' : 'Hebrew',


Mention author.


yes, using ckanapi would remove quite a bit of code


I'll update this to test addrlen against the size of whatever address format that was passed in fi_info when the EP was created.


Not sure I can std::move here. Will take a look.


Mention author


is there a package.json change also?


Will do.


else if ( language == 'hebrew' ) {
// Hebrew *is* Windows-1255
dataBuff = iconv.encode( iconv.decode(dataBuff, 'windows-1255'), targetEncodingCharset );
}
else if ( language == 'russian' || language == 'ukrainian' || language == 'bulgarian' ) {
// Cyrillic *is* Windows-1251
dataBuff = iconv.encode( iconv.decode(dataBuff, 'windows-1251'), targetEncodingCharset );
}


this moves all the typescript language service based processing, right? so that includes generating both the `.d.ts` view and the HTML view of the API?


Add @see to the different exceptions.
Also do that in the method JavaDocs.


" then it will stay for some seconds to let the user time to release button."
I wrote it wrong. It will stay till the user releases the button, If he misses the timeout frame then the ZZ image appears and the radio will not start.


Mention Author.


```
ERROR_NOACCESS
    998 (0x3E6)
    Invalid access to memory location.
```

It's really a memory thing. kErrorAccessDenied is more for file accesses and such for instance.


do we have instructions somewhere saying how to preview the changes to angular.io website given some changes to the TS API?


Makes sense. I'll change that.


JavaDocs.


I'm surprised this is here, since it appears at the bottom of the angular2-specific template. For router it should be declare module "angular2/router"


Move to is's own class.


Does this actually work? Shouldn't this be `0xff & (...)`?


example url changed
Thanks


Alternatively, something like `self.send("set_#{config}", key, value)`?


@rsbondi
about your TODO in L1363, I have the same problem --need look at the generated code :)
In the version of tmpl.js in the tmpl-new-1 brach, I'm using DEBUG to output the code generated for function instantiation here:
https://github.com/riot/riot/blob/new-tmpl-1/lib/browser/tmpl.js#L206
https://github.com/riot/riot/blob/new-tmpl-1/lib/browser/tmpl.js#L244-245
https://github.com/riot/riot/blob/new-tmpl-1/lib/browser/tmpl.js#L329-330

with an option for uglifyjs in the Makefile, this block can be removed automatically, meanwhile, I commented these.


Here is some wrong indentation in that method.


This creates some very elegant test setups!


for the HTML view of the API, we still organize all the classes under the namespace, right? But you no longer need to use `@exportedAs` to determine the correct namespace for each symbol?


Spaces.


Spaces, also move to own class.


Why are you clearing all C debug flags here?


Spaces


Actually this should be 30000. I've forgotten to update it.


did @anurse 's change break the build?


Do I see well that you have fixed that too? Lesson learned: "git commit -a"
does not include untracked files. Maybe add_sourceapp.sh could perform
something like "git submodule foreach 'git add *'" or at least it could
print some reminder message not to forget to add new files (if any were
added) to git for the people like me. :)

2015-07-10 22:28 GMT+02:00 mfonville <notifications@github.com>:

> I now see btw, that this apk is also missing (did not notice it before)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengapps/arm64/commit/4b74e9d4721581f11bc47884e8f46fd0897183a4#commitcomment-12113587>
> .
>


Gold.


Turn on rewrite logging do you see what you really get in apache logs?


Is this LA time?  Why not use UTC?


Does it need to be that "complicated"?  I think that you simply can put `-` in front of `del` and `rmdir` in the macro definitions:

```
DEL = -del /f $(subst /,\,$1)
RMDIR = -rmdir /s /q $(subst /,\,$1)
```


Probably but I couldn't convince myself that nothing else in the project
needed it.
On 10 Jul 2015 22:36, "Alex Eagle" <notifications@github.com> wrote:

> is there a package.json change also?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/petebacondarwin/angular/commit/61c5760c27ea7c0089e419dd016684af2f36b81e#commitcomment-12114739>
> .
>


Please remove


Empty line after this pls.


Hi @mgruner,

Done, thank you!

((enjoy))
cr


Missing LF


Wrong JavaDocs


I feel like *names is a bit of a misnomer, but it is fine for a private API


Remove


Yo, any signs of a new commit?


This should only be the ctrl


I am watching these videos once again and I must say It is not bad as you have done It :) I will leave it up to your decision.


95574472_ip_measure (DavidGoldwasser) - x86_64-MacOS-10.9-clang: Tests Failed

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


this should only be command


Sure


This program simply renders Lineage II map files (well, it did 3 years ago) and the geodata (in l2j format). It does not have any ability to generate geodata. The intent was to eventually add this, but I've moved on to other things.


I don't think you need to check for this condition. `split()` will return a new array of length 1 by default if it's not able to split the string.


Merges pollute the commit log, are you using git pull --rebase?


Don't use .* style imports


It's better to use a Splitter


Please remove these and the other commented out lines of code


Please use the style guide that can be found here: https://github.com/airlift/codestyle


These should go to a log, if needed at all, rather than to stdout


I'm not sure exactly what c is; the convention in the Presto codebase is to use a full word/words


similarly, column rather than clm


You can use multiple catch (e.g. https://docs.oracle.com/javase/7/docs/technotes/guides/language/catch-multiple.html)

Also, to print these errors to the log, the logger you should use is io.airlift.log.Logger


You probably need to update the schemas before any of the public methods (e.g. getSchemaNames, getTableNames, etc), for the sake of consistency


I'm not sold on having cmd in here at all, w/o a way for the user to override it. This _will_ conflict with the multiple cursors feature. I don't use that myself, but I know that people who do are big fans of it.


elasticsearch.metadata.uri would be better


Yeah, you're right. In the latest commit it's 30s. Sorry about this.


@johnmccutchan, what happens now if you ctrl-click in a file on linux? Does it insert multiple cursors?


Is there already a library or method in org.json that would do this?  In any case, this class needs to be renamed and you need to remove the main() class.


confusing naming convention :-) I would name this filenames or filename_list to point out that the data type is list


I checked in a plunk, just stopping the remove in globalEval, but want to have a test case to check automatically


Remove this class or move it to /src/test


@yanex, thanks mate!


All of the existing tests are written with TestNG, it's probably better to use that framework


I was looking for a project like this to add to our continuous integration process.
When I looked at the source code I was pleased to see that it uses spring-batch.
I wanted to use spring-batch and spring-boot for a while but never got to it.
Switching to spring-boot just felt like the right thing to do.
It is my first project with spring-batch and spring-boot.
Hopefully I don't make too many beginners mistakes.


I only got partway through the commit, but it needs some cleanup (especially with exception handling) and tests.


deal with the corner cases here (sys.argv[1] isn't available, or it refers to a non-existent file, or that there are <2 files in the list of filenames)


hey you should hang in #stackvm and #dat and #ipfs IRC on freenode!


Yea I was thinking that would be a smarter way of doing it. I'll fix it when I add some docs in a little while.


Crap.


Ouai j'ai juste clean le code pour récuperer le contexte OpenGL, j'ai pas modifié le nom encore :D


Linked to wrong vlc bug report, correct link:
https://trac.videolan.org/vlc/ticket/15034


just a note: we will eventually output these to binary (hdf5 or similar), so will have to add a "save_to_hdf5()" function here


Csináltam PR-t a hu.po-ra, két módosítással. Az Office az Iroda lett, az Accessories pedig Kellékek.


I guess one downside of this, that our issued tokens don't even contain exp and nbf claims. Now it is so secure that no one is able to login =).


No, we read it when we need it -- Haskell is lazy.
Really it should be `Set QuoteCategory` though, because a `Quote` can't belong to the same category twice.


I will fix that.


Köszi.


:p je m'en doute bien, c'est juste marrant


btw PyCharm will give you recommendations on code formatting (PEP 8)[https://www.python.org/dev/peps/pep-0008/]; which we should follow as much as possible


A genpref pedig alapból noinst, szóval csak a build során használatos.


Could you implement any changes over PR next time? Think community should review changes before they merged to master


You're of course right. I somehow presumed that would interfere with the $(call ...) logic - but after all the whole recipes are just strings ;-)


Eddig benne volt az icewm binárisai közt a genpref.
Viszont nem fut le most a cmake -val fordítva, nem jön létre a preferences fájl, lehet, hogy készakarva?


As I was explaining in https://github.com/yeoman/doctor/issues/22, if it's really gonna work on `0.10.x`, this is a much worse solution than I was meaning to suggest! If you do this, we're gonna have all sorts of issues trying to use Yeoman here, what with production (and thus what we use in development) being `0.10.38`. Specifically gonna run into issues installing `grpc` and other modules that need to be compiled on `npm install`, and won't work if compiled on a different version of Node.


93469234-DemandResponse2 (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1693 of 1693 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)


nit: suggest something like, "Sets the desired time, in seconds, to start playback from."


Hey @ramonskie, would it be cool to relax this one too? Anyone with any constraint on backup (which I apparently have) means this breaks our Berksfile resolution. Any reason it needs to be exactly 0.2.4

(As a rule of thumb for a community cookbook, it would be really really wonderful to not require exact specific versions :)


That field isn't enforced. Just a noop warning. Read the npm docs ;)


This looks like it was a genuine memory leak :cry:


I'm on the road now. Won't be able to look at this till next week. Has this been tagged already?


YOu can write this as

```
match x,y with
    | None, None -> 0
    | Some _, None -> 1
    | None, Some _ -> -1
    | Some(x1,x2,x3,x4), Some(y1,y2,y3,y4) -> ...
```


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


Ok, I couldn't remember. That's good that it's at least consistent. In general tho, don't forgot about us sad, lowly enterprise peeps tho. =) I assure you, I have all my person stuff on Node 0.12 if that makes you feel any better.


New thing you might like ... [ES6 template strings](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/template_strings) by using backticks instead of single quotes:
```javascript
deal.dealText = `Get ${ percentageOff }% off purchase of ${ primaryProductName }`
```


should this just be mscorlib or should it apply to System.Runtime as well when doing PCL compilation?


Looks like you accidentally left a merge marker in.


While you're at it, why not change it to emessage?


Not tagged yet. :)


93469234-DemandResponse2 (rongpengzhang) - 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)


Maybe I need to try recompiling again, but this has crashed twice in 3 hours.


Looking in the global table is not right for any hosted compilation (e.g. Visual Studio project system, FSharp.Compiler.Service, or the hosted compiler used in testing).  I think you'd need to plumb more data about the set of canonical assembly references down (which is not going to be easy to scope correctly).


Use ``not (isMscorlib data)``


Are the getters and setters needed here? Robolectric will call into the underlying class if the methods are not implemented. Same comment for equals.


Is assembly name comparison really case insensitive?  I don't recall seeing that in the ECMA specs but perhaps it is.


This is completely silly because `|` is a valid regex token and also the character limit


use a list here. Sets take longer to construct and in this case there is minimal performance benefit


The test is not running against the DNX build just built on Linux/Mac CI agent.


I think you mean to remove this?


Many thanks friend before sending you mail, this kind of tunnel supports 400 user or more connected or you will have problems?


@CWSpear As I said. It will continue to work on Node 0.10 in the foreseeable future. We're still running tests on 0.10: https://github.com/yeoman/yo/blob/master/.travis.yml

This is just a plot to encourage people to upgrade, as the Node ecosystem have a history of being very slow. Node 0.10 was released on Mar 11, 2013. That's more than 2 years, which is a decade in the tech world. This is just our part in helping Node move forward.


Uh, actually my bad, what I wanted to write is (for a file foo.cc):
1) include "foo.h"
2) C system includes
3) C++ system includes
4) 3rd party
5) own project headers

Seems agree on this one ;-)


JSTL,JSP,Servlet


LIne specific comment


good anyway


The backup constraint  is there because the dependency on fog is really
changing fast. Which means sometimes that the backup procedure breaks.
But I will tests some release and will remove the constraints where I can

Op za 11 jul. 2015 00:05 schreef Patrick Connolly <notifications@github.com
>:

> Hey @ramonskie <https://github.com/ramonskie>, would it be cool to relax
> this one too? Anyone with any constraint on backup (which I apparently
> have) means this breaks our Berksfile resolution. Any reason it needs to be
> exactly 0.2.4
>
> (As a rule of thumb for a community cookbook, it would be really really
> wonderful to not require exact specific versions :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ramonskie/bamboo/commit/cf644bc7ede4d8c88884ba2cde0ae15c60113546#commitcomment-12115239>
> .
>


I'm really not sure of the right place/way to do the unification.  Is this fix just for the FSharp.Core scenario or would it have to apply to other compilations as well?  Doing it when reading assembly refs doesn't really make sense as that makes the AbsIL assembly reading dependent on the set of references - at the moment AbsIL assembly reading is a "standalone" process and no AbsIL metadata is context-dependent like this.

So that feels like an unresolved issue.


What's this?


Lehetséges.
Viszont ha elkészül cmake használatával is, mi az akadálya, hogy akár a compile, akár az install fázisban lefuttasd? :)


This gets the highest ever added to the table, or the current one if its higher, but what about assembly refs created before this one?  Is it ok that they may be lower than the highest eventually found?


There's no R problem you can't solve by just making it into a C problem...


It's a temporary thing, my test is perfect as usual


sorry...I know you changed this before. Must've gotten clobbered


+ optymalizacja w PlayerSyncPosition ---> wysyłanie na serwer tylko w przypadku zmiany pozycji


I totally understand, and my company fully intends to upgrade to 0.12... around a year+ after it was initially released. You should have seen what it took for me to convince then to use `yeoman`. Hopefully this doesn't freak them out =/


What does the Y-Axis account for? Number of cells in the generation?


Also, you need to add the Apache license header to all of the files


This patch needs to be pushed into RHEL7.1+. I setup the LVM logical volume containing the root filesystem  to be using LVM cache and my box failed to boot afterwards.


I can revert this commit if that would help? But we're not dropping the warning in `yeoman-doctor`.


You are awesome @travisleithead ...and fast too.


It would help, and I didn't say to drop the warning, I just suggested we *change* it.


... to try to clarify.  It seems that the version of 'hid_get_feature_report' I was calling added an extra byte as a report ID.  This extra byte made the result 1 byte longer than was expected.  I'm guessing this isn't a widespread problem, or the previous code wouldn't have worked, so I've added my case as an alternative condition.  I don't fully understand the ramifications of this change, but it works in my case.


r+


its return error
`  Warning: file_get_contents(/www/ojs-test/app/../src/Ojs/LocationBundle/Resources/data/location.sql): failed to open stream: No such file or directory`


I mean, I guess at this point, I don't particular care. Cuz on the one hand, the consistency is good and you do make a good point.

I used to think I was indecisive... now I just don't know anymore.


This is good!
But better [HTTP codes](http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html) would be a 404 (Not Found) instead of 401 (Unauthorized) and a 201 (Created) instead of a 200 (OK).
Also, if you want to send data with the HTTP headers, its better to use `res.status(CODE).json(data)` so that its JSON formatted.  In this case, you probably don't need to send data at all but could just do `res.status(201).send()` and `res.status(404).send()` and have the client assume success or failure based on status code.


merging MonsieurNicolas/stellar-core/throttleNomination = 21a6f242 into auto


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


MonsieurNicolas/stellar-core/throttleNomination = 21a6f242 merged ok, testing candidate = 77fa464f


colocar las comillas doble "%....%"


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 131](https://teamcity.labkey.org/viewLog.html?buildId=260307&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 1 Build time: 00:03:48
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest.initializationError: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

```


i will move location.sql to /app/resources/data/location.sql for fix it


`var { id, password } = req.body.passwordResetObject`


fixes issue #30


I don't know, I assume I had problems only with obj.$metadata$


For 1 - Why would the folder already exist but not the assets file? We create that deep folder structure right before creating assets files, and its very unlikely user created same structure in advance themselves. And if they did for some reason have it already, all they miss out on is not having to highlight all and click the sync button themselves to download all assets / runbooks (by the way, we should add to download all runbooks here).

For 2 - We could do that, but I don't think its a good idea. Git doesn't automatically update your workspace code, even if you made no changes locally, to have the latest code. Have to explicitly pull.


@jkeiser A little confused at where config originates and passed into ChefRunData constructor. Can you please help me out?


You removed the `.html` bit


Isn't that what I'm supposed to do?


No - all of our blog posts use the `.html` extension.


@chantastic :beers:


Excellent suggestion! The logic was moved to am_is_paos_request() in auth_mellon_util.c


Done, see above.


It looks like it only needs to write, read and delete one regular file under /tmp/ when logging in, so symlink access isn't required. I tested the change in enforcing more just to be sure, and was able to log-in as admin, root, and an unprivileged user.

The sysfs_t change was an oversight on my part. However, auth works in enforcing mode, so I prefer to leave it out. I defer to you on this, though.


93469234-DemandResponse2 (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (651 of 651 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-5.38%25-green.svg)


OK. I'll look at this first thing next week.


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


95574472_ip_measure (DavidGoldwasser) - 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-NA-red.svg)


Actually (for number2), I'm not sure what the github UI client does..


You've created a circular import here. The `Base` should be in a `meta.py`.


I'm sure this label already exists somewhere


Opps! That was a silly mistake. Good catch. Done.


s/twitter/github/ (in commit message)


gpr_ or GPR_ prefixes on everything as appropriate (but later, once it works)


fixes issue #25


I'd like @jjd-uk to review these strings as well. Seems some could be improved


no:

```
$ ls -l ../test/pw-*
-rw-rw-r-- 1 barb None  18K Sep 27  2014 ../test/pw-cp1251.dic
-rw-rw-r-- 1 barb None  18K Sep 27  2014 ../test/pw-cp866.dic
-rw-rw-r-- 1 barb None  14K Sep 27  2014 ../test/pw-crc32-dup.dic
-rw-rw-r-- 1 barb None  18K Sep 27  2014 ../test/pw-crc32-dup2.dic
-rw-rw-r-- 1 barb None  22K Sep 27  2014 ../test/pw-gen-enc.dic
-rw-rw-r-- 1 barb None  18K Sep 27  2014 ../test/pw-koi8r.dic
-rw-rw-r-- 1 barb None 132K Jan 31 21:38 ../test/pw-new.dic
-rw-rw-r-- 1 barb None  37K Sep 27  2014 ../test/pw-utf8.dic
```

those are valid, and should be version controlled.


please use brackets even if only one line inside. Its hard to read without them


RadianceRefactor (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


You misspelled canola


95574472_ip_measure (DavidGoldwasser) - Win64-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-NA-red.svg)


go right ahead. I assume you can do this easily with some git magic?


fixes #22 #23 #24


For now users can understand what is happening, though this should be localized.


If you want to test, you can build from github..


No it's not needed. It was in the original code I copied so it got pulled in mistake. BTW, the function was renamed am_get_boolean_query_parameter() and made generic so any boolean query parameter could be queried. See above discussion concerning the IsPassive flag.

Done.


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


Which one is the config recommended for production?

This should probably be made clear and easy to change if it needs to be different in production and development environments.


RadianceRefactor (macumber) - 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-NA-red.svg)


I think you'll need to cd into the app directory and do an "npm install" before the cordova build


here you can use `Validate.notNull(clientConfig, "Could not get client because we could not find the cloud that the project was built on. What this build run on Docker?")` to simplify precheck


Cheers!


Not sure about the method being under `ItemEE` but for no it's at least DRY :+1:


Use parallel.Run here, don't invent your own.


I assume `Augments` will have some common functionality, otherwise an attribute might be better. Same for `IAugmentable`.


Now that these checks are centralized here it's easy to change in the future.


what for you return `null` here if later you throw exception on null client config?


Ne pas prendre en compte


Unused!  Gone!


RadianceRefactor (macumber) - Win64-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-NA-red.svg)


@6c3c08e már nem igényli a hu.po patkolást :)


fast-forwarding master to auto = 77fa464f


This fixes #582.


scaffolding ain't great


Er, renamed, rather.


Profil en fr et ez pour rafraichisez.
Je trouve la forme impossible de mettre à jour le profil mieux que "changer".

---- Fix grammar


doesn't give an initial?


This base class support adjusted time, probably should be used in future renderers.


CDブートかつ64bitの場合はループバックの中ではなく外に置いてあるカーネルでブートする。UEFIだとCDブート＋ループバックでI/Oエラーになってしまうため。


The Cream Boston Terrier


This functions and the calls to it look like debug code and should be removed.
If a specific field of levt is useful to log then log it with %x


doesn't add an initial?  just returns name


identation


Not 100% sure but it seems `pgmgr` doesn't handle url in the `.pgmgr.json` file.
See https://github.com/rnubel/pgmgr/blob/master/pgmgr/pgmgr.go#L330


Fixed!


Une erreur de ma part


<br>s extraneous and not best practices?


indentation


I'll try to disentangle it once the timing is fixed.  The problem is the existing mpu401 support was completely entangled with sb16 because of SBMIDI via DSP.


Remove commented out debug code


Urg, sorry, this got moved around a bit and, hmm, the transaction functions don't seem to be tested by the npm tests.  Let me see if this still needs to exist even.

(There is one call site and it does catch the exception...


better practice to find an @users instance variable in the controller and iterate through that then to find User.all in the view (I think)


Buen dia, yo necesito este servicio, me podrian ayudar explicandome como realizarlo?  puedo realizar aportacion por cada caso que se pueda resolver.

Gracias.


displayin password digest?


There is some issues with providing the password in `pgmgr`. It doesn't read `$PGPASSWORD` and it doesn't handle password in url parameter. The password must be set in the password field in the `.pgmgr.json` file.


@eliasdorneles the config recommended for production in this usage with webpack, just deploy the *.js of /static/bundles, not required upload webpack.config.js, but the webpack-stats.json is required, another way the make deploy is create file webpack.prod.js, available example in the next commits


Fixed on ac41d54c6476af920a35bd0b090c5e61aef11868

Overall, the original patch proposed in #2032 has been applied: e9f281f44499c740d2b6631ce8da9f4509f0e9da


[![(Cosmetic box)](http://api.artsmia.org/images/280/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=280)


[![(Writing box with moon and wave design)](http://api.artsmia.org/images/2253/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2253)


[![(Box for writing paper with landscape design)](http://api.artsmia.org/images/2385/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2385)


[![Covered box](http://api.artsmia.org/images/14111/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14111)


[![Covered box](http://api.artsmia.org/images/14112/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14112)


[![(Round Box)](http://api.artsmia.org/images/14113/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14113)


[![(Round Box)](http://api.artsmia.org/images/14114/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14114)


[![(Round Box)](http://api.artsmia.org/images/14116/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14116)


[![(Round Box)](http://api.artsmia.org/images/14120/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14120)


[![Cosmetic case](http://api.artsmia.org/images/34448/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=34448)


[![Round box](http://api.artsmia.org/images/34453/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=34453)


[![Square box](http://api.artsmia.org/images/34454/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=34454)


[![(The Immortal Gama)](http://api.artsmia.org/images/84440/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=84440)


[![Autumn Landscape](http://api.artsmia.org/images/5614/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5614)


[![Pure Sound of the Rivers and Mountains](http://api.artsmia.org/images/5616/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5616)


[![Facing the Autumn Wind](http://api.artsmia.org/images/8557/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=8557)


[![Drifting in a Small Boat](http://api.artsmia.org/images/8716/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=8716)


[![Views of the Xiao and Xiang Rivers](http://api.artsmia.org/images/19784/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=19784)


[![Views of the Xiao and Xiang Rivers](http://api.artsmia.org/images/19785/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=19785)


[![(Handled flower basket)](http://api.artsmia.org/images/60435/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60435)


[![(Cylindrical basket)](http://api.artsmia.org/images/60436/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60436)


[![(Basin-shaped Basket)](http://api.artsmia.org/images/60437/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60437)


[![Tanba-ware leaf-tea storage jar](http://api.artsmia.org/images/2621/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2621)


[![Tea Bowl](http://api.artsmia.org/images/5154/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5154)


[![Tea bowl (chawan)](http://api.artsmia.org/images/5354/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5354)


[![Tea caddy](http://api.artsmia.org/images/9789/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=9789)


[![(Incense Box in the Shape of a Rooster)](http://api.artsmia.org/images/12827/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=12827)


[![Shino-ware square bowl with pattern of hanging vines and fruit](http://api.artsmia.org/images/30352/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=30352)


[![Ko-Satsuma-ware tea bowl](http://api.artsmia.org/images/51415/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=51415)


[![(Incense box with floral motifs)](http://api.artsmia.org/images/60222/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60222)


[![Individual serving vessel (mukÅzuke)](http://api.artsmia.org/images/62356/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62356)


[![(Bale-handled wash basin)](http://api.artsmia.org/images/62657/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62657)


[![Poem Card (shikishi) with snowy willow](http://api.artsmia.org/images/62701/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62701)


[![Two Immortal Poets](http://api.artsmia.org/images/62702/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62702)


[![Takatori-style tea bowl](http://api.artsmia.org/images/62708/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62708)


[![Tobacco tray](http://api.artsmia.org/images/62709/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62709)


[![(Teabowl Stand)](http://api.artsmia.org/images/63501/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=63501)


[![Tea caddy (chaire)](http://api.artsmia.org/images/66031/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66031)


[![Drawstring bag (shifuku)](http://api.artsmia.org/images/66034/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66034)


[![Drawstring bag (shifuku)](http://api.artsmia.org/images/66035/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66035)


[![Drawstring bag (shifuku)](http://api.artsmia.org/images/66036/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66036)


[![(Inner box)](http://api.artsmia.org/images/66038/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66038)


[![(Bowl with Reticulated Designs)](http://api.artsmia.org/images/75318/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=75318)


[![Ido-style tea bowl](http://api.artsmia.org/images/80403/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80403)


[![[Lid for tea caddy]](http://api.artsmia.org/images/80847/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80847)


[![(Fan-shaped Plate)](http://api.artsmia.org/images/81249/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81249)


[![Hotei Within his Sack](http://api.artsmia.org/images/89303/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=89303)


[![Water container (mizusashi)](http://api.artsmia.org/images/97675/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=97675)


[![Water container (mizusashi)](http://api.artsmia.org/images/113511/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=113511)


[![Landscape with Peasants](http://api.artsmia.org/images/1372/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1372)


[![Coastal Landscape with Granite Cliffs](http://api.artsmia.org/images/1645/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1645)


[![The Fugitives](http://api.artsmia.org/images/1241/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1241)


[![The Fugitives](http://api.artsmia.org/images/1310/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1310)


[![Backstage at the ThÃ©Ã¢tre de l'Oeuvre](http://api.artsmia.org/images/1830/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1830)


[![Torso of Adele](http://api.artsmia.org/images/2750/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2750)


[![Door knocker in the form of Medusaâs head](http://api.artsmia.org/images/3189/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3189)


[![L'Amour et L'AmitÃ©](http://api.artsmia.org/images/3276/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3276)


[![The Cathedral](http://api.artsmia.org/images/4194/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4194)


[![Rodin and Chaste Suzanne](http://api.artsmia.org/images/4503/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4503)


[![Bather on a Rock](http://api.artsmia.org/images/4506/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4506)


[![Standing Nude](http://api.artsmia.org/images/4507/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4507)


[![Standing Woman](http://api.artsmia.org/images/4508/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4508)


[![Two Bathers](http://api.artsmia.org/images/4509/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4509)


[![Head of Renoir](http://api.artsmia.org/images/11461/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=11461)


[![(Cosmetic box)](http://api.artsmia.org/images/280/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=280)


[![(Writing box with moon and wave design)](http://api.artsmia.org/images/2253/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2253)


[![(Box for writing paper with landscape design)](http://api.artsmia.org/images/2385/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2385)


[![Tanba-ware leaf-tea storage jar](http://api.artsmia.org/images/2621/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2621)


[![Tea Bowl](http://api.artsmia.org/images/5154/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5154)


[![Tea bowl (chawan)](http://api.artsmia.org/images/5354/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5354)


[![Autumn Landscape](http://api.artsmia.org/images/5614/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5614)


[![Pure Sound of the Rivers and Mountains](http://api.artsmia.org/images/5616/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5616)


[![Facing the Autumn Wind](http://api.artsmia.org/images/8557/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=8557)


[![Drifting in a Small Boat](http://api.artsmia.org/images/8716/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=8716)


[![Tea caddy](http://api.artsmia.org/images/9789/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=9789)


[![(Incense Box in the Shape of a Rooster)](http://api.artsmia.org/images/12827/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=12827)


[![Covered box](http://api.artsmia.org/images/14111/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14111)


[![Covered box](http://api.artsmia.org/images/14112/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14112)


[![(Round Box)](http://api.artsmia.org/images/14113/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14113)


[![(Round Box)](http://api.artsmia.org/images/14114/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14114)


[![(Round Box)](http://api.artsmia.org/images/14116/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14116)


[![(Round Box)](http://api.artsmia.org/images/14120/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=14120)


[![Views of the Xiao and Xiang Rivers](http://api.artsmia.org/images/19784/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=19784)


[![Views of the Xiao and Xiang Rivers](http://api.artsmia.org/images/19785/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=19785)


[![Shino-ware square bowl with pattern of hanging vines and fruit](http://api.artsmia.org/images/30352/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=30352)


[![Cosmetic case](http://api.artsmia.org/images/34448/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=34448)


[![Round box](http://api.artsmia.org/images/34453/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=34453)


[![Square box](http://api.artsmia.org/images/34454/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=34454)


[![Ko-Satsuma-ware tea bowl](http://api.artsmia.org/images/51415/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=51415)


[![(Incense box with floral motifs)](http://api.artsmia.org/images/60222/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60222)


[![(Handled flower basket)](http://api.artsmia.org/images/60435/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60435)


[![(Cylindrical basket)](http://api.artsmia.org/images/60436/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60436)


[![(Basin-shaped Basket)](http://api.artsmia.org/images/60437/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60437)


[![Individual serving vessel (mukÅzuke)](http://api.artsmia.org/images/62356/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62356)


[![(Bale-handled wash basin)](http://api.artsmia.org/images/62657/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62657)


[![Poem Card (shikishi) with snowy willow](http://api.artsmia.org/images/62701/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62701)


[![Two Immortal Poets](http://api.artsmia.org/images/62702/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62702)


[![Takatori-style tea bowl](http://api.artsmia.org/images/62708/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62708)


[![Tobacco tray](http://api.artsmia.org/images/62709/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62709)


[![(Teabowl Stand)](http://api.artsmia.org/images/63501/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=63501)


[![Tea caddy (chaire)](http://api.artsmia.org/images/66031/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66031)


[![Drawstring bag (shifuku)](http://api.artsmia.org/images/66034/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66034)


[![Drawstring bag (shifuku)](http://api.artsmia.org/images/66035/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66035)


[![Drawstring bag (shifuku)](http://api.artsmia.org/images/66036/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66036)


[![(Inner box)](http://api.artsmia.org/images/66038/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=66038)


[![(Bowl with Reticulated Designs)](http://api.artsmia.org/images/75318/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=75318)


[![Ido-style tea bowl](http://api.artsmia.org/images/80403/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80403)


[![[Lid for tea caddy]](http://api.artsmia.org/images/80847/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80847)


[![(Fan-shaped Plate)](http://api.artsmia.org/images/81249/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81249)


[![(The Immortal Gama)](http://api.artsmia.org/images/84440/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=84440)


[![Hotei Within his Sack](http://api.artsmia.org/images/89303/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=89303)


[![Water container (mizusashi)](http://api.artsmia.org/images/97675/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=97675)


[![Water container (mizusashi)](http://api.artsmia.org/images/113511/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=113511)


I think its better to hide the details of HTTP Status codes from the rest of the app. No reason a Store or Component should need to care about the actual error codes, just if things succeeded or failed.  I suggest the following:
```javascript
.end((err, res) => {
  if (err) {
    reject(err)
  }
  resolve(res.status === 201)  // Server should respond with 201 to a successful POST
})
```


The problem with this is that `catch.hpp` is a third party file, so we risk having trouble merging this with the changes to it when we upgrade to a later version. Could you perhaps [submit a PR to Catch itself](https://github.com/philsquared/Catch/pulls) for this?


is it ok to using ocaml-hex like this? https://github.com/sevenEng/ocaml-ci/blob/master/lib/task.ml#L47-L52


97902934_ZoneMixing (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


The test here is for http's XHRBackend, but the change is for the template loading xhr mechanism. I would accept the change in XHRBackend, but the test needs to be changed so that it would actually fail without this change.


Seems legit. Mine was likely working because I did ```make install``` on my libmsgpack build


if you're supporting 1.9.3, this could be the source of a memory leak.


How's that?


Why are you returning a token?  If I understand correctly, the only thing that is changed is the password hash, and that should _not_ be given to the Client.  You could prolly just resolve with whether the request failed or didn't (like my suggest above):
```javascript
resolve(res.status === 201)
```


It's stylized as `C++14`m not `C++ 14`, FYI


I think the error is because this Parallel goes out of scope right away after the setup which then terminates the process, and since you didn't use defer in parallel the callbacks are still invoked when the process is gone and bad things happen!
I think instead of creating a Parallel object, you should follow how collect works which is a method, but spawns a ParallelProcess in there with a Promise returned.
Also how about just making this change into libprocess directly?


Pushed in the wrong branch D:. Meant to create a PR.


Or is this a matter of the test being in the wrong commit?


Did you clang-format?


It could be a DDoS vector if you're using it on unsanitized user input.


@IoTboy FYI, these steps should have been in details-install_xdk.md, not here.


Related to comments on karma_api ... you don't need to store the user/auth data with a password change or the Email HTTP Status Code ... just whether they failed or not.


No, it works fine.


Sorry, the commit message is unclear. Yep, it only surpresses echoing the executed command, not the output. Was more or less just a way to make travis less verbose.


Ah, yes, I see, thanks. Would be nice to commit it separately as this is unrelated to blob SQLite changes.


:heart:


Nit: Should be document in docstring, eg:

state: string, Opaque state string which is passed through the OAuth2 flow and returned to the client as a query parameter in the callback.