Then you can use these two functions to collect the checkin and checkout timestamps, thus can remove the arrivetime and endtime sql columns from the patient_tracker table.


I'd rather make it clear in this place that there is no way init can be FALSE for legacy plugins


See package.json for newer modules.


Thomas I think you should check if the file name already has an extension, before you add it. This way you may end up with double extension.


Are you planning to add an updateFromV6() function? I know it'd require jumping through some hoops, but otherwise people's databases will collect cruft if they installed before this.

Also, don't you need a PRAGMA FOREIGN_KEYS=ON to even enable fkey constraints?


This one too, I think has been broken by SF. Not 100% sure, but I ran it just before sending the pull, and CI didn't complain either.


I don't think you need a refresh due to g != (genBound8 & 0xFC) condition, so you can just write:

  void save(Key k, Value v, Bound b, Depth d, Move m, Value ev, uint8_t g) {

    if (m || (k >> 48) != key16) // Preserve any existing move for the same position
        move16 = (uint16_t)m;

    // Don't overwrite with less valuable data
    if (  (k >> 48) != key16
        || d > depth8 - 2
        || g != (genBound8 & 0xFC)
        || b == BOUND_EXACT)
    {
        key16     = (uint16_t)(k >> 48);
        value16   = (int16_t)v;
        eval16    = (int16_t)ev;
        genBound8 = (uint8_t)(g | b);
        depth8    = (int8_t)d;
    }
  }

But anyhow, I'd strongly suggest to per forma  regression test against current master at STC but with standard hash size.


As discussed above, the arrivetime, endtime, laststatus and lastroom fields get to be removed (ie. these are the crutch data fields that were used to allow development of the feature and are no longer needed :) ).


Several other items on testing listed below:


And it doesn't harm if it's called (again) for the geany package?


The target width and height could perhaps be specified as OptionalParameters? That way the user can produce high-resolution images. Without such parameter the width/height depends on the component size on the screen, which is limited by the screen size.
[I don't know for sure if I understand it correctly... just guessing]


Several other issues noted on testing below:


1. Put Room number directly below status in Calender appt gui.


Printing the stack trace is always good for developers, but perhaps there should be some error message box for the user, since this is a GUI operation?


-Do not show the encounter column in the patient status
-When click on patient in patient status then:
1. Open the patient summary screen in top frame
2. Open the encounter screen in bottom frame (if there is an encounter mapped, then open the encounter; if there is no encounter mapped , then open the past encounter gui)(Basically are doing the same thing that happens when open a patient from the patient search)
3. For those that do not want to leave the tracker when click a patient, provide a toggle that will instead do above step 2 after opening a new complete OpenEMR frame. Basically just need reproduce this from the Patient/Client->Patients gui (note what happends when toggle 'Open in new window' there).


`if (!(flags & TEE_DATA_FLAG_OVERWRITE))` instead?


df


In patient tracker popup, need to populate the Status Type with current status.


Looks good!


In patient tracker popup, if a room number is made blank, it is not recorded (the prior room number remains). Not that this works correctly, though, on the calendar edit_add popup gui.


Review is done :)


commented


The current convention of all glyphs in the font is that the first note is always at the same level, maybe we could keep the same convention here too? That would make things more coherent I believe


Hi, I will test it ASAP. I have found that the whycon does not find a blob,
when it was previously seen (and lost) in other part of the image. It was
resolved by disabeling setting the "use_local_window = true;" to false.
Since then, my only issue is with the input buffer. There is a ~5s delay
befor the blob is found (when not previously seen). But according to the
log, it was actually found in the 1st frame where the blob was present...
So I suppose that there is a buffer between a camera and the detector...
And because locating the preciously unseen blob takes so long, the buffer
fills up...

Thanks, Tomas

PS: Today, I am printing my Masters's thesis ;-).

On Thu, May 7, 2015 at 12:23 AM Matias N. <notifications@github.com> wrote:

> Hi Thomas, I see you changed the way the buffer is cleaned. Did you find
> any bug? or why did you perform the change?
> In case there is something wrong I'd like to fix It in my code as well.
> Thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/klaxalk/whycon/commit/5e6002d7cf2765dc2eda9584a3c6969a899e4a1e#commitcomment-11076382>
> .
>


Now the whole operation is executed in the action listener, therefore it runs on the Swing thread. That means the whole GUI is frozen until the operation is finished. Generally, GUI handlers in java should only contain minimum code and all main operations should be performed on new threads that are started from these action handlers. See e.g. http://docs.oracle.com/javase/7/docs/api/javax/swing/SwingWorker.html


@martynjames `HOMEPAGE_COURSE_MAX` is set to `None` in common.py


I understand that it might be a bit over the top, but it doesn't hurt either, does it? The advantage of bumping the version is that the README in the checkout by Carthage is up to date and the cocoadocs.org docset is also updated.


Looks as in such case if 'use_external_resource' is set we use ip from first connection in list, but this ip can be differ from ip for such node received from management network so will be not available in managment network.


corrected in a later version


What do you mean? Shouldn't it be throwing `NoMethodError`s in the case a particular method doesn't exist?


@martynjames Overrides in devstack.py


Supposedly not in its current implementation. Take look at the main method and add another argument that  does repeat the block which is executed when no filename was given, after the block which does execute the specified script on the pyboard. Call it '--wait' or something.


its new param so by default/in old configuration is not exist - for such case "ctx.node.properties.get('use_external_resource')" will be better.


=D


hm, moving the `.encode('utf-8')` to the return breaks the result (the unicode placeholder is not replaced as it should). Looks like markdown result needs to be encoded in utf-8 before replacing the placeholders.


Yeah, I haven't been considering Windows or CMake. If we're going to go the Cmake route we should go the whole way and drop ./configure completely.

In other news I think this breaks the build entirely and should be dropped. (we should probably enable Travis on this repo --- all the bits are there!)


We should fix cmake the cmake system then. Deleting zconf.h dirties the source tree.


Looks like this is a typo? Snapsho**r**t.


`r'{}'.format(placeholder)` - why that format call? just put `placeholder` there directly. right now you convert it to a bytestring


Change need little fixes and write mock tests for full coverage.


More broadly, we could drop prefix support and clean up the data types and then simply drop zconf.h. That might be vastly better.


これを使うと、各行ごとにコメントできるのかな？


そのようだ


Could we revert this back to 'suite - gcylc[ - host]'? The icon already indicates that it's gcylc, and the suite name appearing first is useful for the panel when you have a bunch of windows open.


What was the reason for this change?


update


似乎预编译失败了？http://sourceforge.net/projects/openwrt-dist/files/shadowsocks-libev/2.2.0-f7da365/ 链接不存在。


Sorry, 'suite [- host] - gcylc' is the current format on master


should'nt featureProperties.toggle (at this point) be always true? or can it be undefined?


True. Putting `placeholder` directly fixed the above problem.


Should there not be any explicit change to the kase object newValue() != oldValue()


Is the mockStrataDataService not working ?


I suggest we keep the test description understandable and elaborate as possible.
'checkForCaseStatusToggleOnAttachOrComment'  ??


I think some of these new cases could be avoided by taking punctum (or quilisma, oriscus, etc.) for ambitus of one? Also the cases with an ambitus of two correspond partly to the new things I wanted to create for gregoria4o (where punctum with a bar has a different width) so we can keep it, but their width here is wrong I believe, the bar is not counted while I believe it should... what do you think? For instance I think you should take `\gre@char@punctum@line@tr` and `HEpisemusPunctumLineTR` (not yet mapped) for case 32, and replace 28 by 6 in `gregoriotex-write.c`. Do you see the global idea? This should make episemus placement a bit better... Sorry you have to deal with this spaghetti plate, I know it's quite uneasy to understand!


makes sense!


commit message


Si i warning sono sulla custom cell dell'imageviewer che al momento devo ancora terminare se da fastidio puoi semplicemente eliminarla tanto al momento non viene usata

Inviato da iPhone

> Il giorno 06/mag/2015, alle ore 21:35, Marco <notifications@github.com> ha scritto:
>
> Giulio possibile che a me dia problemi di linking e qualche warning?
>
> —
> Reply to this email directly or view it on GitHub.
>


Hi there!

I'm glad to see you found my solution. :) I [answered it on StackOverflow](http://stackoverflow.com/a/30094439/4690930), so I hope that helps you out!


Beautiful!


就用這段


Seems ok, I've added test here:
https://github.com/sksamuel/elastic4s/blob/master/src/test/scala/com/sksamuel/elastic4s/query/RawQueryTest.scala

Note that the rawQuery must just be a query not a whole search obviously. If you're still having issues post up your code on stack oveflow and we can help there.


editing...


Dropping prefix support is indeed something I have been considering.


Woah. I didn't mean any harm to ExpAdv2. I think it's a great addition to wiremod. I suppose my joke wasn't expressed too well.


Thank you for that fix!
Why official odometer did not use that?


s/the ongoing fetch/an ongoing fetch/
s/the promise which the function returned/script/

The second substitution is because we don't want it to be observable at all. Otherwise you could use another promise to observe it.

It's also not entirely clear to me we need a new reason, but perhaps it's good to add new reasons as we come across them, as long as they're not exposed to content. Once we start exposing this to content we will need to filter the reasons somehow.


The second line in the commit message should be:

"It doesn't **make sense** to send an email when a user is not logged in."


I see the potential path issue probably crash on windows. Make sure to replace ```/``` or ```\``` depend on user platform.


s/Although the server to which the fetch is connected can observe the termination, the user agent may terminate the fetch/The server observing garbage collection has precedent, e.g. with WebSocket and XMLHttpRequest/


格式有误，需修改


same as above


I think we should find another way rather than listing all incompatible libraries/frameworks.


In particular, "fetch" is not really connected to anything here and we should not use normative language such as "may" in notes.


`if (res!=TEE_SUCCESS && res!=TEE_ERROR_ITEM_NOT_FOUND)` instead


I just pulled, was going to bring it back and found out you were faster than me :) Thank you @aslakhellesoy!


s/may/can/


s/must note/cannot/


Did you mean to link to http://reproject.readthedocs.org/ here?
The way it's not I think it's confusing, because users just reading this line think "the package" is reproject, but then you link to healpy.

Otherwise all looks great to me, :+1: for the release.


Está subido dos veces porque cuando lo subimos hace dos días, github había caido.


s/must not/cannot/
s/termination/as termination/


s/must not/cannot/
s/once/one/
s/to the/for the/


Why?


We should document somewhere that updating bower.json means that we need to update plone bower.json and upload the needed components.


These methods should probably throw UnsupportedOperationException


Now this plugin work in grafana 1.9 only. But I'm working on a new version for Grafana 2.
Did you mean master branch in grafana-plugins repo? I am going to create little web site for this plugin on github-pages with installation instructions, download links and other info.
Thanks for feedback!


why create a 'new' variable?


It seems that a proper fix was written by @jmartinm [here](https://github.com/dset0x/invenio/commit/f28bb643bd9c6dc9cdf2e3db73e0a075403ab01d)

I've undone this change.


Why is this code wrapped within a runnable?


bad idea. if i already have pika installed, this way of installing it (by using more eggs on the PYTHONPATH environment) will create a weird effect - the library will be imported from /usr/lib/python not from your python path. the setuptools.pth file inject site-packages BEFORE the PYTHONPATH


i'm confused. the fix is after 0.9.14, but your egg is named 0.9.14 ... whats going on?


any idea why some tests fail on OSX?


Dobra robota !


use the github search function - "force push" produces plenty of interesting hits. at this point i'm pretty convinced that this is exactly what you need to do (they say that the pull request might start to look a bit funny, but let's not worry about that yet).


Maybe a comma after "what"? The "who is using what we" alliteration makes it a bit confusing.


Bug here - should be #EscapeForJson($v)


It crashed in case Dbic class column had set accessor field. Perhaps accessor should be used instead, but we should think it though. By default accessor is the same as field name, so usually you don't encounter this problem.


you can instead do archive.as(ZipExporter.class).exportTo()


@alex-ab: Works for me


ok


This doesn't work in ReST. It should be

```
``k``\ th
```


:cry: will be fixed in next release thank you for catching this blatant error on my part :+1:


Adjust the indent.


need to seperate the set value in different cases.
1. setBackgroudColor_transparent
2. setBackgroudColor_Color (such as Black, Blue)


这个参数是不是电池容量？不是1930mAh吗？


oh p***... j'ai honte... mais vraiment lol

J'ai fait ce que t'as dit @brinbois encore plus simple même : dans DS tu peux faire "télécharger le fichier torrent d'origine".
Donc je lis l'HTML sans souci mais je l'ai quand même ouvert dans mon navigateur... j'me suis retrouvé avec en gros plus ou moins la page d'accueil de t411 me demandant de me connecter et de saisir un
captcha...

le captcha m'a fait tilt... la dernière fois que t411 m'a demandé ça, c'est parce que je m'étais gourré plusieurs fois de mot de passe... bon vous me voyez venir??? :-( j'ai changé mon mot de passe sur t411 il y a quelque temps... et j'ai oublié de le modifier dans sickbeard... la honte pfffffffffffffffffffffffffffff

sorry pour le dérangement du coup.... #fail


"warm up" usually means performing some initial load in order for JVM to warm up, compile hot methods and fill caches. This method should instead be called something like `serverAwait()`


I grepped through vtk sources and the module's name is vtkIOMPIParallel - but I guess you already knew that ;)


should not combine different value test into 1 case.
need to seperate.


Use t()


Put a comma.


This seems weird. Generic classes should not be extended in a raw way


Is that a typo you left intentionally?


catch on a new line.


xoas di ko can thiet


I see the potential path issue probably crash on windows. Make sure to replace ```/``` or ```\``` depend on user platform.


Rather than changing the signature of the method, it would be better to inject (via a setter) the information as done in `$factbox->useInPreview( $requestContext->getRequest()->getCheck( 'wpPreview' ) );` and leave the `doBuild` as is and adjust the inner workings of the `$this->getMagicWords()` method instead (since this is what actually happens, forcing a magic work state change on an action event similar to the `wpPreview` ).


xoa di


I doublt that you do not use the methods in XWalkViewTestBase.java
setBackgroundColor()
setTransparent()
setZOrderOnTop()


what is the reason for countDownLatch to be an attribute? You can get the same result just with `for (int i = 0; i < waitTime; i++)`


Ok,nhầm


Removing the public function get_categories_posts() (lines 121-124) is causing a problem for people using templates - "Fatal error: Call to undefined method CatList::get_categories_posts()"

See this thread over on the Wordpress Support forums - https://wordpress.org/support/topic/version-06-fatal-error


Did this manually and it worked. thanks.


`@Nullable` missing on parameter because of test `o == null`


`getJarFile2`?
I guess this is refactoring leftover


Can you describe what should I do? Because I don't understand this..


I hear you. This is quite similar to the synchronous step definitions feature request. I'm planning on introducing both soon. There are cases where asynchronous World instantiation makes sense (e.g. initialise your web automation framework, reset DB, etc.), hence the need for this.


Very well then. I'm sorry, I was being overly defensive. Can't tell if it's a joke through text and all that.


just saying before SQ does: key should be final


Do we need the `object &&` check here? Seems when we enter the `do` block, `object` is always not `null` (which should be guaranteed by the condition above -- `if (!object) {`, and by the condition in the `while (object &&`)


a fail-fast null check could be welcome here


extra space before method name


Yeah, this is complicated. Probably we should just pass the environment from where the object was created, and testing for `environment.Object.prototype` -- that would be much cleaner. If it's not possible to pass the environment here, we can go with this yet.


\n


not good.

```php
$this->_redis->incrBy($id, 1); // why not use
```

```shell
127.0.0.1:6379> INCRBY page_view 1
(integer) 20
```


*Did you mean master branch in grafana-plugins repo?*

Yes, here:
     https://github.com/grafana/grafana-plugins/tree/master/datasources/zabbix

Please add link at new Zabbix Share site: https://share.zabbix.com/
Do you have habr account to publish article, I know a lot of people interested in this plugin.


IIRC this removes ability to run buildhck server directly from command line, needing wsgi to run.


I'll apologise too. To @Rusketh, as well as everyone else here. I should've made it clear that it was a joke, within this context.


loggers should be static :-)


No idea, I don't have a machine with OSX, but the tests are currently failing on windows too so hopefully fixing that also fixes those.

Working with filesystems is a bitch because nobody wants to support a platform they don't use themselves.


I think in a class-based approach ```foAI.foAIstate.aggression``` should not be used as a numerical value in functions. Instead, have some appropriately named class attribute that is initially set to the aggression level.

This should allow for easier tweaking in subclasses if we do not want to use the aggression level anymore but still the same formula.


Ah you moved it here.


Codigo antigo, não mostrava o horario na log


"Throws an exception" => "Throws an IllegalStateException"


your code -> your name


Thx ;-)


Redundant code.


Is there reason why these are commented?


Fixed.


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


Looks pretty self explanatory


这个参数是不是电池容量？不是1930mAh吗？


I know the hook uses the context object (it is such a kitchen sink object in MW that I'd rather see its usage banned in SMW as far as possible) but we don't really need the whole object (`$context->getOutput()` and `context->getWikiPage()` ; PS:mocking a context object is a nightmare) therefore the best approach is to narrow it down on what we need.
```
InfoAction(
$context->getOutput(),
$this->context->getWikiPage()->getParserOutput( new ParserOptions( $this->context->getUser() ) ),
$pageInfo
);
```

Constructing an object without additional logic (`if` and `else`) in the callback is what we want to narrow its scope.


There are two events:
`
org.jboss.arquillian.container.spi.event.container.BeforeUnDeploy
org.jboss.arquillian.container.spi.event.container.AfterUnDeploy
`
One of these would probably be a better candidate than AfterSuite


I'm sorry, but I don't see the relation between the commit message and code changes here.


:-1: on this part - we dont want to encourage cloud3 usage anymore


Yes. But let it be as is, I don't want to adjust temporary hack.


really? you don't think making `core` a final property is a good idea? (and create the associated static factory methods)


I writen a sample LRU cache without container/list and committed it into my fork.


Does this commit fixes the crash (#493) in AddFieldTextToHistory?


Nice this is simple and clear :+1:


I don't understand the point of this ``reset()`` at the very end. AFAICT, it needlessly throws out and repopulates the cache for nothing being able to use it before it all happens again.


I would prepend these methods with _selectivity because they are added to the global scope. Or even better I would use an object to encapsulate the behaviour. I'll send a PR


I'm on clang 3.4


This should catch `ClassNotFoundException` only, to avoid unnecessary wrapping of other exceptions (such as `RuntimeException`)


@kirang89 An error occurs when I do this. the explanation below says that stringio object is already an open file object
http://stackoverflow.com/questions/11892623/python-stringio-and-compatibility-with-with-statement-context-manager


I was going to schedule the regression test but see that Lucas already did it :)  Thanks.
http://tests.stockfishchess.org/tests/view/554b0fda0ebc5940ca5d68ea

I'm not sure if you are correct about not needing a refresh.  Your version is functionally different with bench of 8827592.  I'll be happy to test it against the original patch if you like.


모든 form 필드에 대해 clean_ 가 붙인 메서드를 정의할 수 있습니다.

각 필드의 clean_ 메서드가 호출된 다음에, clean 메서드가 마무리로 호출이 됩니다.

clean_필드명 메서드에서 해당 값이 부적합 할 경우에는 forms.ValidationError("에러내용") 예외를 발생시켜주면, html 폼에서 해당 필드에 대한 에러로서 노출이 됩니다.


Fixed #236


@Crayder: No, I didn't test it by myself. I only compiled it on Microsoft Visual Studio 2013 Professional on Windows 8.1 Pro x64 for you. I just compiled what was there on the repository, and the output's size was 1.59mb, this looks pretty wrong, seeing that Incognito's builds for Windows have a size of 200-300kb. I will test it today when I will be home.


What about http://api.jquery.com/ready/ ? I don't think this ``waitDOMReady`` hack is needed.


here is the suspicious return of a non existing directory I talked about earlier


Can you use some `mkstemp`-alike?

There is [0], which also takes care of making sure that it's chmodded 0600 before writing contents to it.

[0] http://ruby-doc.org/stdlib-1.9.3/libdoc/tempfile/rdoc/Tempfile.html#method-c-new


Well, we should drop support for it, then.

Anyway, done.


> I don't get it... is it so that when the original test fail, it returns something like this [ 4, [] ]?

Yes. It also returns [0, []] sometimes unless you adjust the count to 100. The default is 10.

Allright I will make a sub for all *scan tests.


I'm sure the answer is "yes" but I've got to ask: did you think about people manipulating the querystring for nefarious reasons?


This writes contents to the file before it's chmodded 0600, so depending on the umask, there may be a race between write and chmod where the secret can leak to other users on the local system.  Either use something like [0] (mentioned on my other comment) or `fchmod` on the fd returned from the `open` call.

[0] http://ruby-doc.org/stdlib-1.9.3/libdoc/tempfile/rdoc/Tempfile.html#method-c-new


Okay


I really don't like popup windows.. Why don't you make an AJAX request to get the content and then display it in the same page?


There's still "He" at the end of this line.


@benedekfazekas the tests pass on windows now, it would be great if you gave this a try on osx.


`statelessSession` is not closed


It's a great question!! We aren't 'redirecting' per-se, this only sets the 'angular url', not the actual url itself so it's only the path that can change not the host or anything. So you can't be redirected to another site or anything. Pretty sure if you chuck a bunch of stuff in there that it can't route to it will just redirect you back to home and it doesn't render that string anywhere so there's no xss or anything.


please squash this into your first commit so we merge only two


`isHeartbeatEvent` should be in com.github.shyiko.rook.source.mysql.MySQLReplicationStream (as it only makes sense in case of Percona Server and not PostgreSQL, for example)


Good one. Thanks!


``selector.js`` and ``selector.css`` could have been put in a WebJar.


Why not. An iframe in a modal box?


I think something along these commits of vorbis broke something, because libcanberra cannot compile anymore:

```
  CC       libcanberra_la-read-vorbis.lo
In file included from /nix/store/vr4xqb85nvqal7czkixvmrrznkqadbxm-libvorbis-1.3.5/include/vorbi
s/codec.h:26:0,
                 from /nix/store/vr4xqb85nvqal7czkixvmrrznkqadbxm-libvorbis-1.3.5/include/vorbi
s/vorbisfile.h:27,
                 from read-vorbis.c:27:
/nix/store/vv5ynnhx9gvp6yi6ydcagx83790q6s6k-libogg-1.3.2/include/ogg/ogg.h:25:26: fatal error:
ogg/os_types.h: No such file or directory
 #include <ogg/os_types.h>
                          ^
compilation terminated.
Makefile:1396: recipe for target 'libcanberra_la-read-vorbis.lo' failed
```

If vorbis is exposing ogg headers, like it is apparently, it must be a propagatedBuildInputs.


Is this everything


This is amended so it is gender neutral in the new version which will appear after the new government is formed: https://github.com/alphagov/whitehall/pull/2136/files#diff-6ec8282ade015984b9da29753589b3d2R27


missing end tag for trans in search_result.html with cause error when showing search result


Do we have an equivalent to `$xwiki.ssfx.use()` with webjars? If not it's a blocker IMO.


Just curious: what's the reason behind switching to env vars? To me they've always felt rather implicit compared to an explicit properties file.


hhh


Ok! No problem! Volevo fare un po' di test ieri per vedere come stavamo
messi :)

Il giorno 7 maggio 2015 09:28, gsalierno <notifications@github.com> ha
scritto:

> Si i warning sono sulla custom cell dell'imageviewer che al momento devo
> ancora terminare se da fastidio puoi semplicemente eliminarla tanto al
> momento non viene usata
>
> Inviato da iPhone
>
> > Il giorno 06/mag/2015, alle ore 21:35, Marco <notifications@github.com>
> ha scritto:
> >
> > Giulio possibile che a me dia problemi di linking e qualche warning?
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mulasmarco/iOS-RankIt/commit/32abf4f127bd09384d1fd2858d4707e64b8c2bfb#commitcomment-11081206>
> .
>


what can I write?


When exporting existing views, it's recommended to take a look at what changes it makes. This one for example is needed and because your view on local was a little older that this commit, you've removed it.


Making it easier to deploy with docker, and not having this in my source control :)


Very well done job!


> How about adding a simple integrated event loop / scheduler so people won't have to always write their own?

There's a standard event loop/scheduler: http://forum.micropython.org/viewtopic.php?f=3&t=85

> I think it'd be great to have some 3.5 features before CPython :)

Patches welcome, but who needs that, if people don't know/use already existing things?


There's some general background information [here](http://www.paraview.org/ParaView/index.php/Parallel_I/O), but it's very outdated (2007) and doesn't answer **why** (or when) that specific MPI IO module is required. Looks like some of your modules listed above pulls it in, but it in fact is quite a weird dependency.


```
Cloning into bare repository '/home/user/.npm/_git-remotes/git-github-com-am11-pangyp-1b7d331e'...
npm ERR! git clone git@github.com:am11/pangyp Permission denied (publickey).
npm ERR! git clone git@github.com:am11/pangyp fatal: Could not read from remote repository.
npm ERR! git clone git@github.com:am11/pangyp
npm ERR! git clone git@github.com:am11/pangyp Please make sure you have the correct access rights
npm ERR! git clone git@github.com:am11/pangyp and the repository exists.
npm ERR! notarget No compatible version found: pangyp@'am11/pangyp'
npm ERR! notarget Valid install targets:
npm ERR! notarget ["2.0.0","2.0.1","2.0.2","2.1.0"]
```


On my cygwin installation <endian.h> can be found in the
C:\cygwin64\usr\include directory.

You could comment out the <endian.h> and use the following functions
instead of swab_xx provided:

/// @brief Reverse byte order (16-bits).

/**

@param[in] x The 16-bits integer.

@return The reversed 16-bits integer.

*/

inline UInt16 swab_16(UInt16 x)

{

    return (((x >> 0) && 0xff) << 8)

         | (((x >> 8) && 0xff) << 0);

}

/// @brief Reverse byte order (32-bits).

/**

@param[in] x The 32-bits integer.

@return The reversed 32-bits integer.

*/

inline UInt32 swab_32(UInt32 x)

{

    return (((x >>  0) && 0xff) << 24)

         | (((x >>  8) && 0xff) << 16)

         | (((x >> 16) && 0xff) << 8)

         | (((x >> 24) && 0xff) << 0);

}

/// @brief Reverse byte order (64-bits).

/**

@param[in] x The 64-bits integer.

@return The reversed 64-bits integer.

*/

inline UInt64 swab_64(UInt64 x)

{

    return (((x >>  0) && 0xff) << 56)

         | (((x >>  8) && 0xff) << 48)

         | (((x >> 16) && 0xff) << 40)

         | (((x >> 24) && 0xff) << 32)

         | (((x >> 32) && 0xff) << 24)

         | (((x >> 40) && 0xff) << 16)

         | (((x >> 48) && 0xff) << 8)

         | (((x >> 56) && 0xff) << 0);

}

I'm going to commit these changes soon so it'll be possible to select
appropriate implementation via corresponding macro.

2015-05-07 9:31 GMT+03:00 thakorjay <notifications@github.com>:

> hey,
> Can you help me with this where to find this endian.h exactly??
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-11080599>
> .
>


"Creating a network" is somewhat incompatible to distributed computing which I believe to be the basis for Docker, or any cloud. By creating a network, do you mean the old lifestyle of buying a boxes connecting them (I guess you may be thinking of vlan or vpn), with one special box wiring to the internet (I guess you may be thinking of router or gateway)? When everything is connected (when routers are already everywhere), this "one hole to breath air" out-of-date idea is against the very basis of distributed computing. With a network controller (a right step of libnetwork for Docker!), a set of containers can be control-connected on-the-fly in their business need, while each having multiple breathing holes.

Preparing resource before business is the old asset ownership idea and will limit scale-ability of Docker, believe me!

Thanks,
Wenbo Mao
DaoliCloud Company
www.daolicloud.com


This feels less like a shoutout and more like an unofficial deprecation. If E2 is on its way out then that's fine, but we should be clear about that rather than just adding one comment which implies it's the case.


can this crash?


(I mean yes i see the TODO, just thinking about the severity)


oah, it uses SSH if given user/repo...


Yep.


This var19 is the same as var9.  I don't know why CSE isn't figuring this out.


Suggest some better handling. of v_out is actually not v_out. In worst case hide the correction in a.
Function v_out()


use -> user


this relates to #132


来到新东方后第一个小程序，用于爬取网站上的 音标 和 发音。


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


@chelmuth: please add to staging


Why do you want this color?


Because!


这太尼玛危险了吧...


I moved this up from further down. It can't crash because value("key") will return a blank QVariant if the key doesn't exist.


Chuck Norris approves this commit !


Actually I bet I know :(  CSE for us works via the scope tree, which is a conservative approximation of the dominator tree.  We seem to be generating nested terms that "simplify" isn't able to linearize, now that some failures are just continuing to "false" instead of to "fail".  I guess we need to add a pass to translate "false" to "fail" or something.


@taylorotwell is this possible to set within the `phpunit.xml` file? If you would like to fetch it from the environment file instead.


That would be good. The components are updated automatically by running bower update within the static folder in CMFPlone, no?
Maybe we can automate this via buildout.coredev or a simple shellscript: downloading the bower.json for CMFPlone and running bower update afterwards.


Uh, this is interesting and I was actually thinking the same thing the other day.
How comes it never made it into a PR? :D


test comment


`memberShips` with capital S? What kind of vessels does this refer to?


Maybe using logger.exception(...) would be better?


Nice testing of happy paths. Should really also check the sad path i.e. when `is-required="false"` it doesn't set the required attribute. This is how I've structured my tests for datetime: https://github.com/rockabox/rbx_ui_components/blob/matt/9085-datetime-component/test/unit/rb-datetime-control/rb-datetime-control.spec.js#L140


you are using simple present in all other parts. Here you are using simple past. Could you change the tense to simple present. :smile:


Any updates on this?


Oh my, I totally missed they were indexing an array, corrected.
Representing them as bit masks was convenient in debug code of the "full" version (that counted runtime separately), because it allowed me to easily put assertions that thread ought to be in state X or Y in some point, but not Z, e.g. (MONO_PERF_STATE_COMPILE | MONO_PERF_STATE_RUNTIME). It makes no sense to have them as bit masks in this simplified version.


Would be awesome, thanks!


Понял. Тогда предлагаю сделать так:
1. Вернуть поведение LossFucntion к корректному.
2. Добавить к классификатору опцию progressivae_validation_skip - не делать progressive validation по первым значениям. Ибо он довольно бессмыслен - какая разница, как не обученная модель предсказывает.


should be removed or logged an thrown again (because at this point we can't handle the exception with a meaningful warning to the user)


Before merging please check if code compiles properly. I have a problem setting up Anko and compile so I copied the code from my project. Imports should be fine but it's better to double check.


maybe better comment what exit means here


Seems like a lot of work fro something you’d feel is intuitive. :cry:


这里应该是 safe 的（我 PR 文里写的都是 safe），怎么会写成 drop 呢…


I don't think this can be merged yet as I have not seen the behave code for it.


will do


In this version, yes, that was another overlook on my part. In the new version they are always updated when the thread finishes its execution.


Any suggestions?


I suggest changing the meaning altogether.  Instead of LTS = Long Term Support, LTS should mean Long Term Stable instead.  That conveys much better the nature of the long-term release.


Why is lsusb not needed anymore?


>  Pour width et height, je vois pas trop ce qu'on pourrait mettre comme valeur par défaut pertinente

Celles du A4 ?


config 或 data 目录是专用于存放用户数据的，config.js 不属于用户数据；
config.js 是 app 内的配置模块，用来给其他模块提供配置的接口，属于应用组件，
不要放在 config 目录下。


Sorry that merge request wasn't meant to be sent.

Kind Regards,
Edward Wu

? 2015?5?7??18:43?tpor9095 <notifications@github.com<mailto:notifications@github.com>> ???

I don't think this can be merged yet as I have not seen the behave code for it.

-
Reply to this email directly or view it on GitHub<https://github.com/whwong-sit/info9117_flaskr/commit/f244d67de7c6a26f2e17c931b6798c6f5c5b6186#commitcomment-11082261>.


OK, je ne me souvenait plus de ce passage =)


A URI should be used instead of URL.


You should even emphasize 'Stable' when changing that on the docs, ideally explaining the difference between what we perceive as 'Long-Term Stable' and 'Long-Term Support' -- we, the community, do not offer Support for Ceph, but we do offer some of our time to keep a given Stable release alive for a bit longer than anyone should care, both because it's fun and we think it's an important safeguard for old installations.


Wat mooi


@tom2strobl  This is an unrelated problem to your project. Your project hasnt updated because your config says "AudioFX.min.js" while the file is actually called "audioFX.min.js". Thats why the update fails.


Dude this is for cowards, I have my arrow keys mapped to delete


Thanks


I think "and *the* billing you" part sounds wrong.


@ridoo Is this a bugfix or switch to new Java API? I would like to know if I should cherry pick this into develop. I am planning to make a new release today.


i don't know. if no migration was necessary deletion would lead to bad user experience. now it leads to bad user experience if a migration was necessary. but since user (hopefully) know to reinstall a app that makes problems this might be the safer choice


https://robots.thoughtbot.com/always-define-respond-to-missing-when-overriding I overwrote this method and than I found this article and changed to `#respond_to_missing?` - it's the same and additional `class.method(:method_name)`


`if (!o)` instead. Same at other places.


I've used ``$xwiki.linkx.use()``. It's a bit more verbose but it works. See [Tree Macro](https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwiki-platform-tree/xwiki-platform-tree-macro/src/main/resources/Macros/Tree.xml#L322)


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


Fixed in `71081da` :wink:


Should the issue be closed then?


Following function `tee_pobj_get()` should be rewritten using new functions `tee_pobj_find()` and `tee_pobj_add()`


@EHJ-52n makes it more stable as it makes sure that streams are autoclosed onced the end has been reached. In some cases clients (or 3rd party libs) aren't handling this properly.

See https://github.com/52North/SensorWebClient/issues/155


Is it safe to first delete media and then stop?


good point!


https://github.com/soomla/cocos2dx-levelup/pull/8 Resolves the problem. Close.


空白が気になる


Link to store


performBlockOnAllCacheFiles?


Неплохо, но
* Почему то фон с точечками только под шарами, можно его сделать по все


Thanks @chungg I have merged your remarks here: fcd8ebfcc7e74be07d52dba087929d295c59d185


This change looks fishy


ここも空白が気になるｗ


Please note, that libconfig and pkg-config no longer dependencies of bear. Also, there is a newer version, which fix issues (see ChangeLog.md). Thanks for the packaging work!


メソッド名を直したほうがいいです。


i am pretty sure there is no xcode 6.3.1 for Mavericks. Also i really took great care in picking the recommendations. I can assure you that i have tested those combinations with a stock mac. I think they are still valid and would like you to drop the whole readme change from this PR.


Just for completness - i really tested those and they worked ...


Closes #1


I'm afraid this logic isn't good enough, and actually dangerous:

```
$ btrfs sub create test
Create subvolume './test'

$ mkdir test/{foo,bar}
$ btrfs sub create test/foo/baz
Create subvolume 'test/foo/baz'

$ btrfs sub create test/bar/baz
Create subvolume 'test/bar/baz'

$ btrfs sub list -o test/foo/
ID 601 gen 304424 top level 600 path test/foo/baz
ID 602 gen 304425 top level 600 path test/bar/baz
```

This means that *all* subvolumes on a given subvolume will be deleted.

What's more:

```
$ mkdir tmp
$ mount $device -o subvol=test tmp
$ cd tmp
$ btrfs sub list -o tmp/foo/
ID 601 gen 304424 top level 600 path test/foo/baz
ID 602 gen 304425 top level 600 path test/bar/baz
```

This means that the code above will try to delete /test/{foo,bar]/baz subvolumes, assuming that the root subvolume is mounted at /, which is very likely not the case.


В IE не работает салют, и сообщение еще непонятнее
![image](https://cloud.githubusercontent.com/assets/286948/7511832/722d8e30-f4c1-11e4-9619-32f5a75d976a.png)


This comment is pretty short


This breaks parsing colors like `0,0,255,255`


should be "then".


Excellent, that makes sense. Anyway I'll move all validation to the outer *init*, I guess it makes more sense to have it together.


super, tnx


Excellent, that makes sense. Anyway I'll move all validation to the outer init, I guess it makes more sense to have it together.


@Bernhelm can you try https://github.com/sass/node-sass/pull/931 or

    npm install git://github.com/saper/node-sass#notssh


Thanks.


Qui è un UPDATE, l'id non è il riferimento per la riga su cui fare update ?


Could you try and open a pull request? This [document](https://github.com/Homebrew/homebrew/blob/master/share/doc/homebrew/How-To-Open-a-Homebrew-Pull-Request-(and-get-it-merged).md#how-to-open-a-homebrew-pull-request-and-get-it-merged) should be helpful. Thanks!


I'm willing to rename it. Let's wait for the review feedback.


@matthew-andrews ?


can't/shouldn't picOneElement be a private function?


should be polite to our custromers


(same as above) can't/shouldn't getPercentageDecimal be a private function?


Not yet. I will test it more during the week.


Thanks for sharing your code,but when the code comes to "(if(id=glCreateProgram())==0)" (shader.cpp) it shows an error that "0xC0000005: Access violation executing location 0x00000000“. It is seems that you have called the glewInit() already so i do not know what is wrong with the code,look for you help.thanks.


`urlencode` cannot return anything that would need to be `htmlspecialchar`'ed but it also doesn't hurt I guess.


con errores


Shouldn't these at least be inline according to the designs?


Yes, @kfricke's suggestion is a good one.  I should add it to the pyboard.py script, `--follow` would be a good name for the option (like tail -f).


Change this


This is a configuration file, means that the one running the bot is editing this in his own environment.


Ordering


i have changed something


I'm the author of bear, and don't have access for OS X machine. Sorry!


Good point.
For me, it not reproduces in the release build on Windows, even for commits before our fixes.

Will also check release build on OS X later.


I think this link is kind of redundant when "Sign in" is displayed above it (see line 74).


Thanks @marg51 , you're right, but for these examples/demos simplicity, I think it would be best to leave these tests in. If these items change frequently then i'll update it. In a 'real world app' the approach you mention is best. Let me know if you think otherwise or have any other feedback :-)


Smartcover still don't work... And enable modules was not a good idea. I had screen bug after compile kernel with modules.


what happened to `test/actual`? could you re-do/update the pr with just the one line changed for webstorm? thanks


@chelmuth: than please add to staging


Changed the pixel of the footer page  in css


These would be more easily readable if you added a level of indentation between #ifdef/#else/#endif.


try this and fix it.
```
#include "error.h"
#include "error.h"
int main() {

}
```


Totally redundant, but something that some people will look for. :) Also, did you see how many times support us is linked from the home page? :)


I'm mostly on OSX, seems to work here. However, I'm a bit concerned about the nature of this issue.


Hi,
this line causes regression, attributeContains selector (*) matching substring "move_block" will also match the "remove_block" input, and the e.preventDefault() line will prevent block from being removed in the content when remove button is clicked.

Input name in the selector should be prepended with an underscore character, and then both move and remove block actions will work:

`Y.all('#zone-' + BlockDDInit.cfg.zone + '-blocks input[name*="_move_block"]').on('click', function (e) {`


`if(compressed.length < uncompressed.length)` it will send `compressed` with `contentEncoding = null`


follow feature added in 4865a22f788e0997771d15e00d2d895d904eb92a.


```
= link_to image_tag("img-navitop1-on.gif", class: "navitop1", alt: "TOP", width: "40", height: "112", onmouseout: "this.src='http://www.syoujyoskip.com/imgs/img-navitop1-on.gif';", onmouseover: "this.src='http://www.syoujyoskip.com/imgs/img-navitop1-on.gif';"), "/"
``

onmouseとmouseoutの画像パスはhttpから書かずに
/assets/images/〜〜
でok


'./../etc.' is the same as '../etc.' so you don't need the first bit.


Does this work without vtkIOMPIParallel on your system?


Is this a problem in the generator?


@af7 & @bebraw as it turns out the space between the environment variable `TARGET` and the `&&` caused the variable to be `build ` and `dev `, [including an additional space(!!)](https://github.com/remy/nodemon/issues/184#issuecomment-87378478). The following script works and is tested on Windows 8.1, 64 bits, node v0.12.2, npm v2.7.4
````
"scripts": {
    "build": "set TARGET=build&& webpack",
    "dev": "set TARGET=dev&& webpack-dev-server --devtool eval --progress --colors --hot --content-base build"
}
````


Looks like the alpha value is interpreted differently.
int 0-255 vs. float 0-1


It was not great to break semver :-/


Deleting zconf.h is part of bigger change in how the source tree is built... We have to do the cleanup bit by bit so we can be sure it works with both nmake and GNU make, and both with running configure and without running it but using cmake instead. If someone just unzip newer copy of the source tree over old copy, we must detect this as in no case we should require user always has git available. A lot of things in the build system is necessary redundancy to keep files of different build styles in sync.


Ahh I see. Thanks for pointing that out. My bad :)


Maybe it depends on the compiler, which one/version do you use?

Do you build release on OS X using:
make all -B


In my opinion, not much new is happening with E2, but it'll stay for as long as it can. I'll fix it if it breaks etc. It's not entirely deprecated really, but the are more efficient options available now.


It seems it does not work for JS resources.


For delete service key description, do we need to document '-f' option?


I will create an issue now.


Bien cambiado


`fmodf` perhaps ?


话说github还真没有gitlab好用


Why the extra empty line?


Indeed I meant the 'p' ;)

On Wed, May 6, 2015 at 9:36 PM, Kenneth Pirman <notifications@github.com>
wrote:

> I'm not really sure what this means here Vasc :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kenjinp/shooooort/commit/6f9848e25e107e7f94923ce9c9f3b04cb528ccbe#commitcomment-11073147>
> .
>


Est ce que tu as encore ce crash ? Sur quelle branche ?

La backend envoie un format de donnée pas tout a fait défini concernant les schedules, j'ai apporté pleins de correctifs recemment mais ce n'est pas mergé (quoique ce devrait etre sur #47 je crois)


Hi, could you elaborate on what these issues are? I use Backbone.NativeView 0.3.2 (https://github.com/akre54/Backbone.NativeView/releases/tag/0.3.2)  with Exoskeleton 0.7.0 (https://github.com/paulmillr/exoskeleton/releases/tag/0.7.0) and don't seem to have run into any problems.


just something you should take into account, a lot of problems in the long term come from misread requirements. it's more of a mindset than a particular problem with this implementation  :)


@jimaek Thanks for pointing that out, locally the file is already with a capital A, but git on mac doesn't rename the files (http://stackoverflow.com/questions/10523849/git-changing-capitalization-of-filenames), so I had to force it. Thanks!


How does this help exynos devices?.


Some special place in hell must be preserved for ppl who use "l" as iteration variable.


Closes #9
Ref #31


Going to leave this to paul to fix. As there isn't an inline control modifier as far as I can see.


Actually, I used to work with http://api.jquery.com/load/. The problem is that the script is executed asynchronously after the loading of the dependencies defined by requirejs. Because of that, on some devices (iPad), the `load` event was triggered before my script was executed, so the event was never catch by my script.

That is why I have introduced this `waitDOMReady` in some places. Reading the documentation of `ready()`, it seems this event is triggered before the `load` one, so for my use-case it is even worse.


wasd


Nice to see actual code at last


Ich finde 10px ein bisschen viel, 9px wäre besser


I will try to adjust Visual C++ makefile so it can gracefully handle if zconf.h is missing... Currently it still depends on zconf.h.in. I will make pull requests when I finish each part of the cleanup as long as the patch doesn't introduce merge conflict with the current state of the upstream source tree. This pretty much requires me to keep HEAD of my private clone few revisions behind the public one and cherry-picking changes from upstream that don't conflict to separate temporary branches.


This makes a lot of sense; I will do as you suggest.

As a sidenote, operations on immutable datastructures in scala are often just as fast as the corresponding operations on mutable datastructures (in the case of the nonces map here, the performance should be equivalent), see http://docs.scala-lang.org/overviews/collections/performance-characteristics.html for details.


@summerzhao , we have also implemented another two new stories which might affect CLI output a little bit.

1. [CLI user should see a more conventional output format for **list keys**](https://www.pivotaltracker.com/n/projects/1299870/stories/94026928)

2. [CLI user should see a more conventional output format for **get details**](https://www.pivotaltracker.com/n/projects/1299870/stories/94024396)


seems weird to specify a column height as it should inherit table's height, am I missing something here?


@wolfgar I rebased your libcec patch on top of libcec master. does it look good ?


Possibly. I just was following the convention paul's been using. he's not adding `rb` to them.


This was deleted, but the file is still referenced in `functions.php`


Could be probably made static now.


Hey @drewbrokke, looking at FullNameDefinitionFactory I see that it's also reading this same property. In fact it has the exact same code. We cannot allow that duplication. I think we need to get rid of this method and instead add an isFieldRequired() to FullNameDefinition.


Definitely.
Will you look into it?
I can look closer at my project, too - with regard to what is different there from the default.
The above was from "manage.py test".
I am normally using pytest though.


three l's :)


no problem


//The following are ONLY kind suggestions.
//Case-sensitive?
//Does blank space does not mean NO rather YES? Removed.
 CHOISE_YES = set(['yes','y', 'ye', 'YES', 'Y'])


added last


Yujin link got updated


LLVM version 5.1 (clang-503.0.38) (based on LLVM 3.4svn)

Yep, I always use ``make all -j16 -B`` to build the thing.


Why did this happen?
This change prevents sharing code using symlinks (for example, `.d.ts` files), which I think is a perfectly valid use case.


You know I do.


@dnmd Thanks a lot! I'll add a note about this to the book.

Ideally there would be some cross-platform way to solve it but this will do for now just fine. :+1:


Library is from
http://www.codeproject.com/Articles/31702/NET-Targa-Image-Reader


`(#614, )`


What if the denotation becomes `NoDenotation` in  new run an both methods should return `false`?


Are you sure you need the condition here? The codex example is about adding the editor style to just one post type, but this adds it to all post type.


Adapt date?


refactoring ideas (no need for action):

1. `getMetaData` should return a local data type (or we should link against thentos-the-library and use `SessionMetaData` from there).  this would take care of the oddly inlined parser in `handleApp`.
2. `verifyTokenOk` and `getMetaData` do sort of the same thing, and it should be done in one round rather than two.


Any reason not to use vm_record.API.vM_auto_update_drivers here? Would be good to avoid the extra DB RPC, especially since the record is already available.


Configuring the username and password doesn't have any effect on local system I think, no?


Yes, you can just set this property yourself in code.


Remove hardcoded values.


Is there a reason not to leave it in?


Add a log message with the field name for problematic field.


make field accessible using setAccessible(true)


Why not use the old `(XWiki.domIsLoaded && init()) || document.observe("xwiki:dom:loaded", init);` trick (using the [jquery xwiki-events-bridge](http://platform.xwiki.org/xwiki/bin/view/DevGuide/JavaScriptAPI#HBridgingcustomXWikieventsbetweenPrototypeandjQuery))?


Is this class used anywhere?, else delete this file.


during exception files are not closed.


Remove this log statement, it will print whole file.


Extend from ReadOnlyBackup to avoid implementing these method here.


No need to generate query string every time, you can generate a prepare statement during setup.


Remove this comment.


Si on créé un nouvelle mise en page, c'est peu probable qu'on veuille recréer du A4 (inclus par défaut lors de l'install)


This does not test the operator or its functionality? Is it required?


Did you mean to remove this? It's used all over the place.


Probably shouldn't be called `test` :) (below too)


Bardzo ładnie, załóż pull request do dev-model.


Je vais corriger le commentaire ligne 228, pour indiquer pourquoi le fromat doit être en portrait


Hard coding this into the app worries me a bit. Is there a better way to define the relationship between the API key and the content?


These commands aren't run in `:privileged` mode, though. Are you sure that it worked before the sudo change?


can you remove the empty defs?


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


I'd still leave it like this. In a sense, isTerm, isType, isClass are defined on Symbols, not denotations. So they should be independent of the current denotation of a symbol. A symbol is a trem or a type even if does not have a denotation in a particular period.


can you add licence?


can you add licence


Hi John, I think it duplicated in line  93. There is another operation_not_allowed with the same value.


What is this? my eyes hurt a bit :) Is this getting cached?


I fixed everything as suggested (see two following commits), I'll look into the tests later.


You might want to use the horizontal ellipsis char here. `…` or …


switch/case. no?)


Are you sure that's also the case in templates?  The knockout docs frequently use observables in templates without calling them (http://knockoutjs.com/documentation/visible-binding.html).
I saw the error on the locations list page, and I reproduced it just now, but I think it was an issue of browser caching, as I was able to get it working.  Sorry about that.


@pierreinglebert: do you have any idea why this fails with

```
> node-zopfli@1.3.3 install C:\Users\xmr\Desktop\node-zopfli
> node-pre-gyp install --fallback-to-build

C:\Users\xmr\Desktop\node-zopfli>if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" clean )  else (clean)

C:\Users\xmr\Desktop\node-zopfli>if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" configure --fallback-to-build --module=C:\Users\xmr\Desktop\node-zopfli\lib\binding\node-v14-win32-x64\zopfli.node --module_name=zopfli --module_path=C:\Users\xmr\Desktop\node-zopfli\lib\binding\node-v14-win32-x64 )  else (configure --fallback-to-build --module=C:\Users\xmr\Desktop\node-zopfli\lib\binding\node-v14-win32-x64\zopfli.node --module_name=zopfli --module_path=C:\Users\xmr\Desktop\node-zopfli\lib\binding\node-v14-win32-x64 )
Traceback (most recent call last):
  File "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\gyp\gyp_main.py", line 18, in <module>
    sys.exit(gyp.script_main())
  File "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 534, in script_main
    return main(sys.argv[1:])
  File "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 527, in main
    return gyp_main(args)
  File "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 503, in gyp_main
    options.circular_check)
  File "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\__init__.py", line 129, in Load
    params['parallel'], params['root_targets'])
  File "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\input.py", line 2698, in Load
    targets = BuildTargetsDict(data)
  File "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\input.py", line 1294, in BuildTargetsDict
    target['target_name'],
KeyError: 'target_name'
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\configure.js:343:16)
gyp ERR! stack     at ChildProcess.emit (events.js:110:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:1074:12)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "node" "C:\\Program Files\\nodejs\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "configure" "--fallback-to-build" "--module=C:\\Users\\xmr\\Desktop\\node-zopfli\\lib\\binding\\node-v14-win32-x64\\zopfli.node" "--module_name=zopfli" "--module_path=C:\\Users\\xmr\\Desktop\\node-zopfli\\lib\\binding\\node-v14-win32-x64"
gyp ERR! cwd C:\Users\xmr\Desktop\node-zopfli
gyp ERR! node -v v0.12.2
gyp ERR! node-gyp -v v1.0.3
gyp ERR! not ok
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute 'node-gyp.cmd configure --fallback-to-build --module=C:\Users\xmr\Desktop\node-zopfli\lib\binding\node-v14-win32-x64\zopfli.node --module_name=zopfli --module_path=C:\Users\xmr\Desktop\node-zopfli\lib\binding\node-v14-win32-x64' (1)
node-pre-gyp ERR! stack     at ChildProcess.<anonymous> (C:\Users\xmr\Desktop\node-zopfli\node_modules\node-pre-gyp\lib\util\compile.js:83:29)
node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:110:17)
node-pre-gyp ERR! stack     at maybeClose (child_process.js:1015:16)
node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:1087:5)
node-pre-gyp ERR! System Windows_NT 6.1.7601
node-pre-gyp ERR! command "node" "C:\\Users\\xmr\\Desktop\\node-zopfli\\node_modules\\node-pre-gyp\\bin\\node-pre-gyp" "install" "--fallback-to-build"
node-pre-gyp ERR! cwd C:\Users\xmr\Desktop\node-zopfli
node-pre-gyp ERR! node -v v0.12.2
node-pre-gyp ERR! node-pre-gyp -v v0.6.6
node-pre-gyp ERR! not ok
Failed to execute 'node-gyp.cmd configure --fallback-to-build --module=C:\Users\xmr\Desktop\node-zopfli\lib\binding\node-v14-win32-x64\zopfli.node --module_name=zopfli --module_path=C:\Users\xmr\Desktop\node-zopfli\lib\binding\node-v14-win32-x64' (1)

npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Program Files\\nodejs\\\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "i" "--build-from-source"
npm ERR! node v0.12.2
npm ERR! npm  v2.7.4
npm ERR! code ELIFECYCLE
npm ERR! node-zopfli@1.3.3 install: `node-pre-gyp install --fallback-to-build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the node-zopfli@1.3.3 install script 'node-pre-gyp install --fallback-to-build'.
npm ERR! This is most likely a problem with the node-zopfli package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-pre-gyp install --fallback-to-build
npm ERR! You can get their info via:
npm ERR!     npm owner ls node-zopfli
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\xmr\Desktop\node-zopfli\npm-debug.log

C:\Users\xmr\Desktop\node-zopfli>
```


This is very bad idea. H4X0rs actually use `sl` and its goal is to 'cure your bad habit of mistyping'. Mistake should never happen for them.

> sl is a highly advanced program for curing your bad habit of mistyping.
> -a     An accident is occurring. People cry for help.


Thanks @saper but I'm building node-sass as a dependency for grunt-sass (v1.0.0) which uses node-sass 3.0.0.

I'll downgrade grunt-sass to v.0.18.0 which depends on node-sass v.2.0.0 until this is stable.

I think some of your issues are related with this error, as https://github.com/sass/node-sass/issues/918
as long as you cant build node-sass as a dependency, it will show this message:
```
Error: `libsass` bindings not found. Try reinstalling `node-sass`?
```


发送到


Ah OK, never mind then :)


Puppet


Hier soir je l'avais a chaque premiere ouverture, puis apres ca ne le faisait plus (on resume) -> crash a chaque lauch donc, sur la branch 47. je reessaye en rentrant chez moi.


Oh, yes, I will modify that. Thanks.


This is still incomplete, there are more explicit paths used in e.g. https://github.com/bflad/chef-docker/blob/master/providers/container.rb#L467


Some minor coding standards issues with variable names in this function.


Simple oversight. Fixed in https://github.com/git-for-windows/git/commit/27ae7e2b403634b8055758776badab851b88900e.


The naming doesn't follow the other methods in the class. I'd prefer something along:

on_error


I would probably expect that we call the error handler here too.


You can use the PHP date object in a more OO manner and get the values without doing explode.


@jurgenvinju can you have a look at these tests? It seems that the standard type reifier builds incorrect `cons` values.


I wonder if that could be done a bit nicer, e.g. by giving an array of possible paths to the command and having it return the right one. L86-L95 would then become

```ruby
chkconfig = @system.check_requirement(["chkconfig", "/sbin/chkconfig"], "--help")
```

We could then maybe rename the method to `find_command`?


>So they should be independent of the current denotation of a symbol.

But with this change it's even more dependent:
```
val wasTerm = sym.isTerm // returns true, as symbol was defined in in previous run
sym.exists() // denotation has changed to NoDenotatioj
assert(wasTerm == sym.isTerm)
```
Now double checking could be required.
Though from less forcing point of view I see the motivation, but that one more detail one needs to know.

Can we make this behavior a fastpath, guarded by a `isDefinedInCurrentRun` check?


Wouldn't it be better to keep this note in (or an equivalent one, to explain why `SENDFILE_BACKEND = 'sendfile.backends.simple'` is necessary)? It's not unthinkable that someone might want to use both medusa and a non-simple sendfile backend, so it's better to explain what their options are, rather than just flatly telling them to use sendfile.backends.simple - particularly if this is going to be fixed within django-medusa at some point.


SwipeRefresh Layout comes from v4 package


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


@gdelhumeau I've used RequireJS (in a JSX object) but I understand why you can't do this. We need ``jswx`` and ``sswx``. Shouldn't be hard to add.

@Enygma2002 because ``XWiki.domIsLoaded`` is set by Prototype code. We want to move on. New JavaScript code should depend as little as possible on the old Prototype code. jQuery's ``.ready()`` should work just fine.


Ok, ça me va comme point de vue =)


the import cold simply be omitted.


On Thu, 7 May 2015, Bernhelm wrote:

> Thanks @saper but I'm building node-sass as a dependency for grunt-sass (v1.0.0) which uses node-sass 3.0.0.
>
> I'll downgrade grunt-sass to v.0.18.0 which depends on node-sass v.2.0.0 until this is stable.

3.0.0 is stable, if you don't rely on custom importers or functions.

> I think some of your issues are related with this error, as https://github.com/sass/node-sass/issues/918
> as long as you cant build node-sass as a dependency, it will show this message:

No, this is a different issue.

If you can't test my patch, then I would suggest just to upgrade your pretty ancient npm,
or double-checking its configuration.

Something there assumes SSH connection should be the default for GitHub.


Yes, the resend requests are still sent shortly after the first one and there is no rate limit, that's something that would be nice but not critical so I'll postpone it a bit. Actually after merging the NTP fix there is another problem, which is that NTP timing information needs to be requested periodically as well.

The solution to all this would be to put a timeout to the RTP thread select and use it to process events i.e. every second. This can in turn be used to process this kind of periodic events like resend and NTP requests. But first I have to get the output thread finished and then add the AAC code and fairplay handshake of yours, that should get the audio working pretty ok.


@gdelhumeau they say:

> If .ready() is called after the DOM has been initialized, the new handler passed in will be executed immediately.


Duplicated `private`


Yes, good idea.


Changing a text's length is impossible, right now it can be change with a hack, but please don't as it will make the job of partial buffer updates impossible. Create the text object with a string of the max length you will ever need, then you can replace it with a smaller one but not a larger one so it fits in the same memory space.


```
$ mocha test/support.js test/API-loopback-token-searchDefaultTokenKeys.js
  API:Middleware:loopback.token(options)
    options = {}
      V Test header=authorization and expect 200 (43ms)
    options.searchDefaultTokenKeys: [true|false]
      V Test header=authorization searchDefaultTokenKeys=false and expect 200
      V Test header=authorization searchDefaultTokenKeys=true and expect 200
      V Test header=authorization searchDefaultTokenKeys=undefined and expect 200
  4 passing (104ms)
```


Not sure I follow. I believe Felicity specifically asked for FE cache to be added for the live site.
If this is not needed on staging, shouldn't the Puppet config be changed?


This is the cell height (row height if you prefer). I agree, this should probably go somewhere else...


@chilsou : ici on récupère l'évènement "click" et on décide de stopper son comportement par défaut ( puisque c'est un clic sur une ancre, on ne voudrais pas qu'il revienne à ancre vide `#` qui est le haut de page.

Pour bien faire, il faudrait que chaque lien pointe vers une ancre posée sur l'élément que l'on va faire apparaitre. Niveau accessibilité.


This is RDBMS-specific (= non-portable). rook-target-hibernate4-cache is only allowed to use Hibernate API (just like original PrimaryKey.getColumnIndexByNameMap).


Fixes #153


@EricTAnderson Just putting out there that the while loop downstairs makes sure that checkleg always gets called at least every half second (that's cautious timing, which we'd be in if we've arrived), is your concern that that isn't fast enough?


This function doesn't have the right signature. A C API function should have a simple signature like `jl_simplevector_isdefined(jl_svec_t *v, size_t i)`, and checking only needs to be done at the julia interface.

But, I think it would be better to implement this entirely in julia in base.jl as a method of `isassigned`.


GSP code shouldn't be touched, since it should be considered a replacement for the official GSP binary rather than part of the emulating code.


RadianceRefactor (macumber) - Win64-Windows-7-VisualStudio-12: OK (2183 of 2200 tests passed)

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


All the changes here are to notify the backend of texture invalidation, which has to be done here. (Since it's where the cache flushing calls are.)


@coolsquid that is only for the current GuiScreen, so if I switched from MC to another app it wouldn't copy over. This is the system clipboard and I set it with no owner (gui) so it will transfer over ;)


fishtest does not do PGO builds on the Windows builder, as explained by Gary. Only Linux and MinGW workers are using PGO. Test deleted.


Using this https://github.com/garyvdm/dulwich/commit/062bb810b96d8ddce290026a8d897d1a56eb5d5a to print out protocol data and git output,

I get this on windows:

    GaryvdM@GARYVDM-WIN ~/dulwich (debug_protocol)
    $ /c/Python34/python.exe -m unittest dulwich.tests.compat.test_client.DulwichTCPClientTest.test_fetch_pack_no_side_band_64k
    [4508] Ready to rumble
     --> b'0036git-upload-pack /server_new.export\x00host=localhost\x00'
    [4544] Connection from 127.0.0.1:49516
    [4564] Connection from [::1]:49517
    [4564] Extended attributes (16 bytes) exist <host=localhost>
    fatal: The remote end hung up unexpectedly
    [4564] Request upload-pack for '/server_new.export'
     <-- b'00d3514dc6d3fbfe77361bcaef320c4d21b72bc10be9 HEAD\x00multi_ack thin-pack side-band side-band-64k ofs-de'
     <-- b'003f35e0b59e187dd72a0af294aedffc213eaa4d03ff refs/heads/branch\n'
     <-- b'003f514dc6d3fbfe77361bcaef320c4d21b72bc10be9 refs/heads/master\n'
     --> b'0063want 35e0b59e187dd72a0af294aedffc213eaa4d03ff multi_ack_detailed ofs-delta multi_ack thin-pack\n'
     --> b'0032want 514dc6d3fbfe77361bcaef320c4d21b72bc10be9\n'
     --> b'0032want 514dc6d3fbfe77361bcaef320c4d21b72bc10be9\n'
     --> b'0000'
     --> b'0009done\n'
     <-- b'0008NAK\n'
    E
    ======================================================================
    ERROR: test_fetch_pack_no_side_band_64k (dulwich.tests.compat.test_client.DulwichTCPClientTest)
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "c:\Users\GaryvdM\dulwich\dulwich\tests\compat\test_client.py", line 227, in test_fetch_pack_no_side_band_64k
        refs = c.fetch(self._build_path('/server_new.export'), dest)
      File "c:\Users\GaryvdM\dulwich\dulwich\client.py", line 233, in fetch
        progress)
      File "c:\Users\GaryvdM\dulwich\dulwich\client.py", line 553, in fetch_pack
        proto, negotiated_capabilities, graph_walker, pack_data, progress)
      File "c:\Users\GaryvdM\dulwich\dulwich\client.py", line 429, in _handle_upload_pack_tail
        data = proto.read(rbufsize)
      File "c:\Python34\lib\socket.py", line 374, in readinto
        return self._sock.recv_into(b)
    ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host

    ----------------------------------------------------------------------
    Ran 1 test in 1.375s

    FAILED (errors=1)

and this on Linux:

    garyvdm@garyvdm ~/dev/dulwich (git)-[debug_protocol] % python -m unittest dulwich.tests.compat.test_client.DulwichTCPClientTest.test_fetch_pack_no_side_band_64k
    [2081] Ready to rumble
    [2082] Connection from 127.0.0.1:36141
    fatal: The remote end hung up unexpectedly
     --> b'0036git-upload-pack /server_new.export\x00host=localhost\x00'
    [2081] [2082] Disconnected (with error)
    [2083] Connection from [::1]:36035
    [2083] Extended attributes (16 bytes) exist <host=localhost>
    [2083] Request upload-pack for '/server_new.export'
     <-- b'00c9514dc6d3fbfe77361bcaef320c4d21b72bc10be9 HEAD\x00multi_ack thin-pack side-band side-band-64k ofs-de'
     <-- b'003f35e0b59e187dd72a0af294aedffc213eaa4d03ff refs/heads/branch\n'
     <-- b'003f514dc6d3fbfe77361bcaef320c4d21b72bc10be9 refs/heads/master\n'
     --> b'0063want 514dc6d3fbfe77361bcaef320c4d21b72bc10be9 multi_ack_detailed multi_ack ofs-delta thin-pack\n'
     --> b'0032want 514dc6d3fbfe77361bcaef320c4d21b72bc10be9\n'
     --> b'0032want 35e0b59e187dd72a0af294aedffc213eaa4d03ff\n'
     --> b'0000'
     --> b'0009done\n'
     <-- b'0008NAK\n'
    [2081] [2083] Disconnected
    error: git-daemon died of signal 9
    .
    ----------------------------------------------------------------------
    Ran 1 test in 0.228s

    OK

Hmmm, actually, this is the first time I've compared the 2, and I think I see the problem.


You can use std::bind or a lambda function instead of introducing a new function here.


@mikeloz2015 done with my review. checkpoint complete. press on!

let me know that you get these comments by responding in github.


NP.

I actually just thought of a cleaner way to fix this.


It doesn't make sense to show the enter key to unlock your phone, but this key is needed to unlock the SIM card. This commit should be reverted since there's no SIM lock check.


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


This is needed because the actual XML response has a "quota" element which contains several more elements.

As such the output of get_user is like `{"enabled": "yes", "quota": {"total": "4298348", ... }}`


No. If I recall correctly, it's the exact code as you're using there. Works for me.


What does the real GSP binary do when you request a cache flush?


can point to his post?


Weird, I'll look into it, I said all that based on what the method looked like considering it is called from the GUI screen :laughing:


develop (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1136 of 1136 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)


Seems to be another case of Mojang having the wrong places for things...recommiting


Eh, I see the point. It's somewhat annoying though since it makes the renderer rely on the particular GSP implementation (especially: HLE vs. LLE GSP).

I suppose at least there should be a TODO to move those NotifyFlush calls to a common function which would be called whenever a service module flushes a cache (including the official GSP module, if we were to LLE it, so it would likely be a function within the CPU core).


I hate being stingy, but I'm torn between it being a good (but forgotten) reminder if it's in latexml.sty, and feeling obligated to keep it working the same way later when a proper solution might be developed.

It *is* a rather simple bit of code that could go into a safeio.sty.ltxml that you could preload yourself and be more sure of it being loaded in the correct order?


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


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


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


use unique_ptr here


@Subv: It... does a cache flush. :D

The thing is in the 3DS the CPU and GPU share texture memory, so to upload a texture all you need to do is write to it in RAM. On a PC we need to detect when that happens and trigger a re-upload of the texture to the GPU.

@neobrain: Agreed, it's probably best if it's indirected through a common function so it's easier to integrate in other uses in the future.


Actually, use unique_ptr in general whenever you allocate memory on the heap and free it in the same scope anyway.


ResortedSurfaces (amirroth) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1138 of 1138 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.84%25-green.svg)


FYI, `3.0.0-rc4` and `3.0.0-rc5` are still listed in npm as of today.


message received! pressing on.


Or preferably, don't constantly re-allocate it. :P


WSHP_SHR_GreaterThan_1 (Nigusse) - 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)


WSHP_SHR_GreaterThan_1 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1644 of 1650 tests passed)

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


Ah, sorry, this should be Stdout.


Once you give this the go ahead on this and my or Nocholas' pull I'll close out my fork on phonegap.


fixed


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


Don't abbreviate variable names like that. Instead, keep the names close to the definitions in pica.h


The `<` at the beginning of the line should be kept. If you're not already, make sure to edit the docs using `conceallevel=0`. I use this mapping, although there's probably many ways to do this:
```
nnoremap <silent> coc
      \ :set conceallevel=<C-r>=&conceallevel == 2 ? 0 : 2<CR><CR>
      \ :set conceallevel?<CR>

```


Why is this virtual? Is there any reason anyone would ever want to extend this class?


What's the reason for having this interface?

Also, documentation plz.


This never gets deleted, does it?


well, shit.


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


IMO, all of the changes in CANhub and PDM made the formatting worse lol


To be honest I was a fan of the more descriptive names, but this is fine too!


What is the point of swapping if the values from `stamp_reg` and `stamp_sec` are never read from?


Should this be `partner.uri`?


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


Can we change the log message to something like `Not reporting metrics to Kafka due to missing Kafka configuration(s)`?


LGTM :-)


LGTM other than the minor comment.


Instead of enclosing the whole function body, just return early here if color_buffer==nullptr


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


` delegate {};`


Sure, no problem ☺

Khajonpot Sawangdee
Senior Product Engineer
Tax & Accounting, Government
Thomson Reuters
Phone: +66 (0)2-648-9510
Cel: +66 (0)90-971-2484

khajonpot.sawangdee@thomsonreuters.com<mailto:khajonpot.sawangdee@thomsonreuters.com>
thomsonreuters.com
tax.thomsonreuters.com/aumentum

From: KevinFaussadier [mailto:notifications@github.com]
Sent: Friday, May 8, 2015 12:01 AM
To: blueimp/Bootstrap-Image-Gallery
Subject: Re: [Bootstrap-Image-Gallery] Use the https version of the demo. (b8b3ad2)

Hello,

If I want to add an icon to download the picture next to the "Play" one in the borderless template, how can I do that @blueimp<https://github.com/blueimp> ?

Thank you

—
Reply to this email directly or view it on GitHub<https://github.com/blueimp/Bootstrap-Image-Gallery/commit/b8b3ad2e5864402c5aae88b51120014061d95b19#commitcomment-11091654>.


There are functions in video_core.h (I think) which deal with this kind of conversion, so please re-use those instead.


A while back we discussed exposing statistics like memory usage through a dictionary, and I thought things like that could go here. Then the way those dictionary values are determined internally would vary by API, each having their own implementation  of ResourceManagerBase. Alternatively this could be axed and statistics and such could be stuck in the renderer base class instead.


See https://github.com/mjohnston/react-native-webpack-server/issues/15 . Can you be more specific about what breaks?


The OSX stat option `-f` is the equivalent to the stat option `-c` on Linux.


Could just memcpy strlen(p)+1 to store trailing null character instead.


Might want to default to true?


Variables like this are bad style and lead to confusing code. This kind of stuff should be handled by a state manager instead of being handwaved by the renderer itself.


@pieper Make sure to fork DCMTK and publish your topic. The following url does not exist: http://github.com:pieper/DCMTK


Ah, no, these are aggregate functions not keywords. They're named in lowercase.


Short array syntax here:
[$currency_code]


Thanks for separating new and call tests


This is bad style. Resources should be initialized when the module is being initialized, not at some random point after that.


.. nevermind, ignore that comment.


If only English had actual gender neutral pronouns.

Or grammar nazis allowed the English language to change over time and let us use "them".


Why would this not be the case? This looks like a bad workaround for improper usage of this function. The improper calls to this function should be fixed instead.


Yup, "Cannot invoke waitForExpectationsWithTimeout with arguments ({Double})". *shrug*


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


Wow, english!

 fixed single-digit positive temperature bug and cleaned up case 2.


Tracking resource state like this is bad style. Why not have the resource manager deal with this stuff?


You Changed to 2.12.81 but...

     const WHATSAPP_VER_CHECKER = 'https://coderus.openrepos.net/whitesoft/whatsapp_version

Here its showing 2.12.73

Pls reply


@Shauren, You are awesome!


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2177 of 2200 tests passed)

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


Why does this need to be a separate function outside of the constructor?


Ah, the reason: the silly DB layer is defined as returning an `Int`, and I didn't want to futz with that unnecessarily.


Why is this part of the public interface? Sounds like something which would only be called internally, and upon initialization.


Can you define more precisely what "about to draw" means?


Can you define more preceisel what "a copy" is and what "about to happen" means?


This is the case when it is the first frame being rendered (no commit necessary) or a NotifyPreCopy is called (notified via gsp_gpu on display transfers and DMA's) before the first frame. Since I'm not 100% sure on 3ds internals and when those could trigger, I err'd on the safer side by checking every time. An alternative would be a flag indicating if the first frame has rendered or not, but it would function in the same way.


started working on the last two enemy's


I suppose it should be the callers responsibility to always pass a physical address here.


develop (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1139 of 1139 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)


Yeah me too. We don't have a well established convention, so both ways are fine.


WSHP_SHR_GreaterThan_1 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (626 of 626 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.28%25-green.svg)


It feels a lot more messy to have random style files created for no good reason then it is to have one (or two?) strategic styles that accept latexml options. You don't have to feel obligated to keep it in later, we can simply remove it if we have a similar/better workflow. That's what major versions are for, LaTeXML 0.9 isn't that far away.


I think it would be better to incrementally track register changes, rather than translating over the whole pica state over to renderer state all the time.


This needs to trigger after OpenGL context has been set up so OGL calls can succeed, but this rasterizer is created before then.


What is this for? Please add a comment to explain the interface, and its properties.


I'll fix some of the issues I find but I'll still mention them here for the record..

Looks like atypo on the return for the "is_throttle_mix_min" method.  It should return a bool instead of a float.


addObject() checks whether the array already has same item or not with contains() before pushObject().
but this is not proper way for grid-nx.
just pushObject() without existing check!


This sounds very vague. In which direction does the synchronization happen? Will the FB in emulated RAM be overwritten with the OGL FB's contents, or vice-versa, or possibly both ways?


No docstrings


r+


Forgot to remove this declaration, oops.


No se encontró el archivo


Seems like a problem even before this patch but it's confusing to have functions and methods with the same name but very different purposes like we have for the "update_throttle_thr_mix".  That's both a function in copter and a method within the AP_Motors library.


están


thanks bb


Favor de dar creditos con CREATIVECOMMONS  checar la licencias en https://stateof.creativecommons.org/


This shouldn't be removed.  You need to free the wallet


Actually is does exist - but it is private because it is forked from
Michael's private repository (it's not actually secret, just not yet
official so he doesn't want it to circulate until the API is finalized).
He says he will be merging it to dcmtk master before project week.  I just
pushed this into these experimental branches to keep track.

On Thu, May 7, 2015 at 8:41 PM, Jean-Christophe Fillion-Robin <
notifications@github.com> wrote:

> @pieper <https://github.com/pieper> Make sure to fork DCMTK and publish
> your topic. The following url does not exist:
> http://github.com:pieper/DCMTK <https://github.com:pieper/DCMTK>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pieper/dcmjs/commit/b1c3f6b878a61315a5d7f01cf7ea2fa6cba3eb84#commitcomment-11099091>
> .
>


How do you propose to test first-run consistency against regressions?


These four variables are also things which should be handled in a standalone state manager instead.


tev stands for "texture environment". The current variable name suggests that this is an array of 6 texture environments, which it definitely is not...


I have no idea sorry.. I stopped working on this years ago when I accidentally broke my phone.


:smirk: :thumbsup:


Can you define more precisely what this means?


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


merging MonsieurNicolas/stellar-core/protocolCleanup = d6be8b40 into auto


MonsieurNicolas/stellar-core/protocolCleanup = d6be8b40 merged ok, testing candidate = 917cfab7


So from now on, just listen to me, hm? :P


Hi @simonjohansson,

Unfortunately, these changes worked their way in by a pair working on our CI pipeline who did not have any context on your PR. As a result of their work, they did the quickest implementation possible to fix our issues. Sorry this happened, it was unintentional and we'd like to use your changes instead (we still don't want any defaults in the spec).

Sorry again,
@DanLavine && @utako, CF Runtime Team


Shut up :P


I pushed a first pass at making these serializations customizable on a [branch](https://github.com/bluesliverx/grails-spring-security-oauth2-provider/commit/f2260612f7f6bcfc1c199d530c6ff7627c076bf7). The default serializers behave how the plugin does now.

If a consumer wants to customize the approach used, they would need to provide a custom implementation to handle the serialization and override the appropriate bean.

For example, storing access token scopes as a `String` and serializing the `additionalInformation` fields as JSON:

```groovy
package test

import grails.plugin.springsecurity.oauthprovider.OAuth2ScopeSerializer
import org.springframework.security.oauth2.common.util.OAuth2Utils

class StringScopeSerializer implements OAuth2ScopeSerializer {

    @Override
    Object serialize(Set<String> scopes) {
        return OAuth2Utils.formatParameterList(scopes)
    }

    @Override
    Set<String> deserialize(Object scopes) {
        return OAuth2Utils.parseParameterList(scopes)
    }
}
```

```groovy
package test

import grails.plugin.springsecurity.oauthprovider.OAuth2AdditionalInformationSerializer
import groovy.json.JsonOutput
import groovy.json.JsonSlurper

class JsonAdditionalInformationSerializer implements OAuth2AdditionalInformationSerializer {

    @Override
    Object serialize(Map<String, Object> additionalInformation) {
        JsonOutput.toJson(additionalInformation)
    }

    @Override
    Map<String, Object> deserialize(Object additionalInformation) {
        new JsonSlurper().parseText(additionalInformation as String)
    }
}
```

And the domain classes would change slightly to reflect the new storage:

```groovy
package test.oauth2

class AccessToken {

    String authenticationKey
    byte[] authentication

    String username
    String clientId

    String value
    String tokenType

    Date expiration
    String additionalInformation

    String scope

    static hasOne = [refreshToken: String]

    static constraints = {
        username nullable: true
        clientId nullable: false, blank: false
        value nullable: false, blank: false, unique: true
        tokenType nullable: false, blank: false
        expiration nullable: false
        scope nullable: false
        refreshToken nullable: true
        authenticationKey nullable: false, blank: false, unique: true
        authentication nullable: false, minSize: 1, maxSize: 1024 * 4
        additionalInformation nullable: true
    }

    static mapping = {
        version false
        scope lazy: false
    }
}
```

```groovy
package test.oauth2

class Client {

    private static final String NO_CLIENT_SECRET = ''

    transient springSecurityService

    String clientId
    String clientSecret

    Integer accessTokenValiditySeconds
    Integer refreshTokenValiditySeconds

    String additionalInformation

    static hasMany = [
            authorities: String,
            authorizedGrantTypes: String,
            resourceIds: String,
            scopes: String,
            autoApproveScopes: String,
            redirectUris: String
    ]

    static transients = ['springSecurityService']

    static constraints = {
        clientId blank: false, unique: true
        clientSecret nullable: true

        accessTokenValiditySeconds nullable: true
        refreshTokenValiditySeconds nullable: true

        authorities nullable: true
        authorizedGrantTypes nullable: true

        resourceIds nullable: true

        scopes nullable: true
        autoApproveScopes nullable: true

        redirectUris nullable: true
        additionalInformation nullable: true
    }

    def beforeInsert() {
        encodeClientSecret()
    }

    def beforeUpdate() {
        if(isDirty('clientSecret')) {
            encodeClientSecret()
        }
    }

    protected void encodeClientSecret() {
        clientSecret = clientSecret ?: NO_CLIENT_SECRET
        clientSecret = springSecurityService?.passwordEncoder ? springSecurityService.encodePassword(clientSecret) : clientSecret
    }
}
```

And in resources.groovy:

```groovy
import test.FooBarTokenEnhancer
import test.JsonAdditionalInformationSerializer
import test.StringScopeSerializer

beans = {
    // Other beans here

    clientAdditionalInformationSerializer(JsonAdditionalInformationSerializer)

    accessTokenAdditionalInformationSerializer(JsonAdditionalInformationSerializer)
    accessTokenScopeSerializer(StringScopeSerializer)
}
```

Obviously the documentation would need to be updated to explain this in a bit more detail, but I think this would satisfy the need to have more control over how these fields are serialized.

What do you guys think about this approach?


I think `return a - b;` is supposed to be better


LGTM, well done.


I noticed that I am a master holder. I'm not sure it is correct behavior or not.
I apporogize in advance, if I made mistake.


i feel like this should just be
authorize @comment since we are already authorizing post above


should this be

if policy(@comment).create?


I'm pretty sure there is a better way to do this. Converting all 150+ timezones to there local time and checking if it is within the 6th hour does not seem too efficient but i could not come up with a better way to select the correct time zones that also accounts for daylight savings time.


so why did we change the routes around here?  i think we want to leave them as they were before and then adjust the routes accordingly for the rest of the stuff...unless I'm missing something...lets chat :).


ok nevermind....im wrong looks like this has changed a bit. so you are right and good to go :+1:


Is there a way to re-use the CSV column headers somehow? We're currently repeating the header logic in 2 places and it might easily get out of sync


ah ok - yes i love this good work


Umm.  _WOW_.  I was wondering what you were up to!


typo here


Same story here, thank you VERY much!


https://github.com/Shougo/unite.vim/issues/905#issuecomment-98795805

But I will add the option for it.


エスケープに弾かれる如何にかかわらず、viewに関わる記述（htmlとか）それそれ以外の場所に書くのはやめたほうが良いです。

また、改行を表現したいだけであればbrタグではなく、divやpなどのブロック要素でうまく配置しましょう。


need to return here


no need to use in this step cos not yet to create user


should remove the ccomment


not yet use sessionHelper should remove


Should the series and events have a level as well?


In the static pages no need SessionController to create. Remove.


Needs JSDocs


delete no need to use yet.


Strikes me that it might be saner to have the op at the top of the stack since it's a global ops order


"was broken ,try" to "was broken, try" :)


Change the word "sample app" to name of our project "Framgia training system"


Here is the main app.js code, I mentioned earlier that there were 3 parts, but I just cut them off since it would just be too tedious for you. It has all the code mentioned in the presentation and will run, but depending on available memory you may be only able to run most of the code and not all of it.


Should we not be verifying the metadata values?


should remove the comments #


Hi @dolfelt - I can't tell from the code, but does Vero support multiple groups?


yes, my bad


元のコードをコメントで残している理由はありますか？
ないのであれば消しましょう。履歴はgitに残っています。


Shouldn't ```sparse``` be set to ```true``` before ```ko.utils.extend``` is invoked? Otherwise it would be impossible to set it to ```false``` using ```options```.


@bbeaird They don't support the native group object, so this will append the Group object to the user under a `group` property where they can be accessed. Not ideal, but works.


thanks a lot
At first sight, it looks just fine yes
I will test asap and report back after real testing...
Regards


should remove this file.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1138 of 1138 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)


lolololol A+


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


@chrisfoster22 @Rocky-R
There's a typo in this SQL, a missing space: `ANDcontent IS NULL`


OpLogger? OpsLogger? ThingThatLogsOps?


Shouldnt it be stmt = CharacterDatabase.GetPreparedStatement(CHAR_DEL_CHARACTER_GARRISON_BUILDINGS);


Is this the same semantics as before the pull request?

On Thu, May 7, 2015, 3:22 PM Craig Tiller <notifications@github.com> wrote:

> We ref() all slices contained in the message, so it's ok to unref() them
> after AsyncMethod, but only safe to change them after the completion of
> Finish().
>
> On Thu, May 7, 2015 at 1:06 PM Vijay Pai <notifications@github.com> wrote:
>
> > What does this imply about when it is ok to change the contents of the
> > buffer that was input to the AsyncMethod? Is it immediately after the
> call
> > to AsyncMethod or only after the call to Finish ?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ctiller/grpc/commit/3676b38183c2d288c6daef1c0f3fd9b08f3927bc#commitcomment-11095322
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ctiller/grpc/commit/3676b38183c2d288c6daef1c0f3fd9b08f3927bc#commitcomment-11097504>
> .
>


must remove the  "<title>Fels</title>
- <%= stylesheet_link_tag 'application', media: 'all', 'data-turbolinks-track' => true %> " cos you repleace it by "<title><%= full_title(yield(:title)) %></title>
+ <%= stylesheet_link_tag 'application', media: 'all',
+ 'data-turbolinks-track' => true %>"


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Thanks. I merged these in and refactored them into their own classes. I also noted that the accelerometer didn't have its pin set, so fixed that.


Well, I want to improve the message.
But my English skill is very limited.
Can you improve the message?


想定される利用人数的に大した問題はでない**かも**しれませんが、基本的にループの中でクエリ（SQLの処理）を発行するのはご法度です。
これだと`max_day`変数の数だけクエリが発行されてしまいます。

クエリは重い処理です。簡単にボトルネックになりますので、ここら辺は手癖にしておくと良いです。


fast-forwarding master to auto = 917cfab7


`extend` will overwrite properties in the target object. So I need to set `sparse` after calling `extend`.


no need to push this file .


88737258-EPW-Ruby-Access (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (2186 of 2204 tests passed)

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


@sanemat pushed!


You should add "CONFIG_BU52061NVX=y"