A brief series of experiments should suffice to establish the optimum timeout.


On peut se passer de cette ligne, N=4*T est déjà fixé par défaut dans fill_bank_spec.m


The old profiles are compatible.


byteSuffix is needed in order to get the value like that - "2.45Gb", if it false we get - "2.45G". If you pass in arguments - (1023, false) or (1023, true), you get 1023B in both cases.


Yes I think so too.

But there is a bug in the online API which I have reported already, the 'Modified' property has a difference of 2 hours and a few ticks between Insert(entity) and Get(entity).

Modified=/Date(1437472065325)/ < insert
Modified=/Date(1437479265500)/ < get

So beware of this untill the API is fixed


@64FF00 Can you revert this because git messed up some things.


Not sure what the root cause is, but this change has caused resrcify (which is later down the browserify transform chain) not to be able to locate and rewrite assets. Still checking


OK, are there additional places where faces and lines are interpolated like this? We should update the code appropriately


Range 3/range 5 is really high on a lot of maps. It of course depends on the metagame - whether people use wraparound maps and how many players they typically have - but for a lot of mid-sized games range 3 on a dual-wrap map may be enough to make a handful of spells hit anywhere, and range 5 would probably pull that up to close to half. While that's not necessarily a bad thing (even if it would make range boostering items totally superfluous, as well as other mages or other sites with range boosts), having that for every path seems excessive - I could see that as being a consideration up there with units or mages for picking a nation on certain fairly common maps. For #allrange it might be better to tone it down - maybe to 1 and 3 or 2 and 3.


![](http://s24.postimg.org/i3uboijed/Untitled.png)

Huh. So it is different for illuminati.


Does it work if you replace `el` with the `$el` defined on L12?


Don't you need to prevent `transitionTo` from happening if there is an error creating the list?


¬_¬


Next time create a second commit for this extra changes please.


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


...and/or possibly scrap allrange in favor of elementrange and sorceryrange, though those could be pretty powerful too, depending on the map.


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


Yeah, sorry. Simple fix I figured out shortly thereafter. Although, still not getting the first test to pass. Not sure what's up.


LGTM


Thankyou! Could you bump your `npm version` and publish?


Yes, I will correct that to true. Ted also pointed out the same thing.


Hmm, that's no good. Guess I will revert.


...was there a reason water was only range 2?


IDK about adding adverts....


I've been trying to figure out why Firefox OS requires you to set `font-family` globally like this (`*`). do you know why? or do you know whom I should ask?


The main difficulty with moving to more recent Java versions is that the bytecode parser we use doesn't support the more recent instructions, and therefore can't parse the JDK `.jar` files included with those versions. I definitely think we should update the parser, but it's a non-trivial amount of work.


@kidoman I just published 3.1.5 that reverts this change.


I commented out the indent_size in 24436bc, letting user decide (I usually use 4). Since we're using tabs for indentation, we can leave it that way and most people can be happy.

There is a downside, and it's things like
```
#define DIGEST_SIZE                             16
#define BINARY_SIZE                             4
#define BINARY_ALIGN                    4
#define SALT_SIZE                               0
#define SALT_ALIGN                              1
```
These should be lined up using spaces, not tabs. The problem is there doesn't seem to be support for enforcing spaces in editorconfig (or even emacs). But I can live with this.


Any version will work the same way as we are ultimately using pbuilder to create a base chroot images of distributions for which we want to build our packages


It is amd64 according to pbuilder's manual


Curious: if you are linking with libwebp.lib, then why is there a new version of libvpx at http://windows.php.net/downloads/php-sdk/deps/vc14/x64/ since the day before yesterday?

@cmb69 Which libwebp version are you using? 0.4.3 are just master?


pff


probably needs `{}` on that `if` statement, no?


Thanks for doing this, Tiffany!


@eteq - I really would like to thank you for the reply. I'm Github newbie and I didn't know it the "@" operator would tag you in here, I was think it should just create a link to your github page to remember me of put the credits.

You're right, I should probably use the astropy. I'm new with Python and Astrophysics to tell you the truth. A friend of mine recommend me this function because she uses in her work and I like it because looks like I could implement a parallel version using the Apache Spark framework.

I will try a version using the Astropy and then try understand how it's implemented. Thank you!


nevermind, i see youve fixed it :)


ha o'ziz yozishiz kerak. ba'zilarda har bir ma'nosi har xil so'z turkumiga tegishli bo'ladi, shunga ma'nolarni tagida yozish kerak.


Not sure what you are trying to achieve with this....


keyin, katta harf bilan yozilmasligi kerak. atoqli ot bo'lsagina katta harf kerak


And even more curious: if I make a beta2 snapshot without libwebp.lib, but with vpxmt.lib available, php_gd2.dll reports: WebP Support 	enabled. See
https://phpdev.toolsforresearch.com/php-7.0.0beta2-Win32-VC14-x64.htm


Looks unused.


Still missing call to completion block


Minor formatting notes: You only need the first `|` for Haddock, so omit the others. Also, use `checkedIopub` or `checkedIOPub`, since everywhere else is snakeCase.


I don't feel 'default offset' conveys the fact that it isn't a valid offset at all.


Thank god


Can you rework this to avoid partial `read`? Instead use `readMay` or similar `String -> Maybe Int`, and then propagate errors upwards and use `Maybe` monad bind or applicative to construct values.


Avoid underscores in variable names – the rest of the IHaskell source code sticks with snakeCase, so just for consistency.


snakeCase


This tool is MUCH more limited, than a full source style enforcer, BUT it should fix a lot of stuff.

No, I think you can still do this:
```
<tab><tab>else if (something() ||
<tab><tab>........something_else()) {

```
The .'s above are spaces, and this tool does not convert that to tabs (at least the little testing I have seen).
Time will tell, but having this thing trim spaces for me auto, and always making sure a new file has a trailing lf, and that all files are in lf format should help quite a bit.


this should probably just be a `throw`ing function, instead of using exceptions


No me gusta el uso de chdir porque uno suele olvidarse de restaurarlo si hay excepciones


getting an **unknown alias lookup: quickeditmenu** on startup


It does slow down my editor (and whole computer) some. We will have to see how much.


autofocusはスマートフォンでバーチャルキーボードが強制的に出してしまうので、使いどころを考える


This does seem to argue for making `cryptol-server` a separate package. The `libzmq4` dependency is particularly a pain since it's a C library, and not necessarily installed on every system, so making it a default dependency would significantly complicate builds. From a quick look a the patch, it seems that the changes to the core Cryptol code (in prover invocations, etc.) could stay in the `cryptol` package, but that the stuff in the `cryptol-server` directory could go in a separate `cryptol-server` package. Do you think it's that straightforward, @acfoltzer, or is there some complexity I'm missing?


I don't really understand why this is called `withDynamic` – that to me implies use of `Dynamic` for dynamic typing.

Actually, now that I think about it... how is this different from just using `serveProfile`? Aren't all of these port values already existing in the `Profile` data structure in `Types.hs`?


Falta una a prueba para no existe. Es preferible no usar fs.exists (leer la doc de esa función).

Y si se usa hay que controlar lo que devuelve


ip (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

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


Does anyone what's the release date for Spring Security 4.0.2 ?


:smile:


r+


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/50070ad94e880c17ef8bab7a021ed00de7cbe855


merging graydon/stellar-core/bug-647-delay-snapshot-write-during-catchup = 50070ad9 into auto


graydon/stellar-core/bug-647-delay-snapshot-write-during-catchup = 50070ad9 merged ok, testing candidate = 7a9e1ecc


El local config hay que leerlo una sola vez al iniciar el modulo. No en cada run


Testing comment


@traversaro, I got all the information from @mikearagao. I'll look the sea_joints branch by @hu-yue and let you know. I hope to get Vizzy's eyes working in the near future.


Thanks.

Just one question.
 Is eval-when-compile compatible with emacs24.3? That's the minimum version we need support.


Oh, that's a good point. I should add a `return` there.


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


Fixed: 9b6f0f2677fa782b14a2684f1261ca5ab322a730


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


Merge sha 7a9e1ecc is stale.


graydon/stellar-core/bug-647-delay-snapshot-write-during-catchup = 50070ad9 merged ok, testing candidate = 2ee44180


Needs echo statement.


ip (axelstudios) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Does this check if pointer has empty char stored?


I'd rather see all of this happening inside of `lib/imap_notifier/config.rb`, I think?

What are your thoughts on creating a `/tmp/imap_notifier.d/` directory, then populating PIDs in there?

I think maybe this weekend I'll spend some time refactoring a lot of this module, I think it's do for a full overhaul anyhow.  If you find a DRY way to achieve what you're after I'm all for implementing it now though, "this weekend" always seems to turn into 2 years from now.


ip (axelstudios) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


if you support 5.6, you have to revert or amend e1b0ebca3d2b079176b12c87e74d0a5fac6c79b1


Needs an echo


Hi brady
Tested as:
Select patient  -> select encounter ->  Fees _> Fee Sheet    =   Fee Sheet Form Shows Up
Modified  /interface/main/left_nav.php  line 1338 to  load_form.php?formname=fee_she”et
   =  Fee Sheet Form Des not Shows Up


@yajra can you add the try catch again? I'm having this issue on some cases:

local.ERROR: exception 'Symfony\Component\Debug\Exception\FatalErrorException' with message 'syntax error, unexpected ';'' in /project/public/laravel5/project/vendor/yajra/laravel-datatables-oracle/src/yajra/Datatables/Helper.php(75) : eval()'d code:1

The $parsed_string variable content is:

```
<a href="<?php echo e(URL::to('admin/language/' . $id . '/edit' )); ?>" class="btn btn-success btn-sm iframe" ><span class="glyphicon glyphicon-pencil"></span> <?php echo e(trans("admin/modal.edit")); ?></a>\n
                    <a href="<?php echo e(URL::to('admin/language/' . $id . '/delete' )); ?>" class="btn btn-sm btn-danger iframe"><span class="glyphicon glyphicon-trash"></span> <?php echo e(trans("admin/modal.delete")); ?></a>\n
                    <input type="hidden" name="row" value="<?php echo e($id); ?>" id="row">
```

Thanks


getHashTable = H.new


No, we will migrate from HashSet to MapSet only with Elixir v1.2 which will be OTP 18 only.


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


pourquoi


@tobie, one more thing. The body of the issue doesn't seem to be filled in with the issue content. On issues-notes.js:104 you look for `ghIssue.body_html`, which doesn't seem to be present and is not a documented feature from the GitHub API. There is a `ghIssue.body` attribute, however.


I like the goal of making the tests more portable, but this change switches to an absolute error magnitude. I'm pretty sure 0.001 is a very significant error for some of the conversions. If I neglected to include some tests with very small numbers, that's a more important problem to fix.

Maybe we could take the ratio of the expected and actual values?

It might be most robust just to fix the scientific notation differences.


fast-forwarding master to auto = 2ee44180


I reverted this change and pushed the copyright changes and the other minor changes that I did.
I will work on some of the other issues as I get a chance but most of what I left is things that will require an answer from your team.
Thanks
Terry
      From: Ramesh Nagul <notifications@github.com>
 To: ensoftek/openemr <openemr@noreply.github.com>
Cc: teryhill <teryhill@yahoo.com>
 Sent: Tuesday, July 21, 2015 1:24 PM
 Subject: Re: [openemr] Merge of AMC changes from DrCloud into OpenEMR (170.314(g)(1) and (g)(2)) (1de5353)

Hi Terry,

The change we put in was very specific in nature to make it work. The existing Encounter count results in false positives.

Anil/Vamsi, can you comment and make sure this works?

Best regards,

Ramesh Nagul

Technology

(m) 503 816 3867

(w) 503 643 1226 x 118

(f) 503 626 1769

 <http://www.ensoftek.com/> www.ensoftek.com

 <http://www.drcloudemr.com> www.drcloudemr.com

Please be aware that e-mail communication can be intercepted in transmission or misdirected. Please consider communicating any sensitive information by telephone.. The information contained in this message may be privileged and confidential. If you are NOT the intended recipient, please notify the sender immediately with a copy to <mailto:hipaa.security@ensoftek.com> hipaa.security@ensoftek.com and destroy this message

From: teryhill [mailto:notifications@github.com]
Sent: Tuesday, July 21, 2015 9:51 AM
To: ensoftek/openemr <openemr@noreply.github.com>
Cc: Ramesh Nagul <ramesh@ensoftek.com>
Subject: Re: [openemr] Merge of AMC changes from DrCloud into OpenEMR (170.314(g)(1) and (g)(2)) (1de5353)

added this

$options = array( Encounter::OPTION_ENCOUNTER_COUNT => 1 );
if (Helper::checkAnyEncounter($patient, $beginDate, $endDate, $options)) {

and removed

 $sql = "SELECT count(*) as cnt " .
 "FROM `form_encounter` " .
 "WHERE `pid` = ? " .
 "AND pc_catid = 5 ".
 "AND `date` >= ? AND `date` <= ?";
 $check = sqlQuery($sql, array($patient->id, $beginDate, $endDate) );
 if($check['cnt'] > 0){

—
Reply to this email directly or view it on GitHub <https://github.com/ensoftek/openemr/commit/1de5353c59e4fbab992ebdb5f8202eddab991f9d#commitcomment-12279916> .

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


So much damn boilerplate


Sorry for the rerun episode of "Drive-by Commenter" :-P

But is this meant to address the "cognitive dissonance" that occurs when training the SP and TM at the same time? Just curious...


Oh, I see, I made the original error!

You just removed the hash lookup and sent the expected value in to the conversion test harness. I like that change.

It seems like since this code is being touched again, we should fix the underlying fixed error magnitude problem.


Thanks, Done


!tasks.Any() で例外出す


Yes.


Only 104 need it.


Ok. Will revert this commit. Scrutinizer is suggesting that the try catch is unnecessary but it seems that this is needed. Releasing this on v5.8.5. Thanks for reporting!


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


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


It just replaces if (!sql) where sql is char* to handle std::string cases too.


By the way, I think the test results were automatically generated with a grep command. I see the --R-- tag matches the test output, so it seems like I just ran the tests against GNU units and then fixed the code until things passed. I must have made enough iterations to make the --R-- tag worthwhile!


That's odd. I tried it with Emacs and it merely adds a way to tell Emacs a few of the countless settings.


Yes, but that's an orthogonal issue, qt4 is irrelevant to my question (nothing should be loading both qt4 and qt5 libraries in the same process).


@richardjgowers : I used you as the committer because you've been responsible for most of this


Let me know if you find anything, I'm curious.


*spank*


but im not sure that !sql is the same as sql != nullptr.
I mean, does !sql check if it's empty?


I think we should simplify the packages available in the base lockfile to only those that are strictly necessary or universally desired.


Using a notification here because when opening a tab from the BrowserViewController, we have to wait for the TabTrayController to appear before opening a new tab.


Why do this?


This seems extraneous?


Corrected spelling error in register.h
```c
struct _virutal86
// to
struct _virtual86
```
Also. Adding in cert signing into cert.h.

NDLL exports code also being written in.


testFunc() is temporary, it's just to help you understand the flow.


this isn't an ideal way of handling errors.. either you create your own exception class or throw the exception and let the calling entity (class, method, etc.) to handle it..


No it doesn't, it just checks if the pointer is null. if(!sql) <-> if(sql == nullptr).


I'm sorry, but I don't understand the question. What's the problem that needs solving?


:fist:


You're seeing it in the content-type header or elsewhere?


I was hoping sth like `if ! [ -e "$f" ] || deps/jlchecksum "$f"; then <download again>; fi`


You mean I can simply assign null to it? I thought it would be better if it is nonenumerable.


that's a good idea, go for it


The strings that served as keys for your intent "firstNum",  "secondNum", "operand" should be declared as a constant variable and be used all throughout the application.. That way it would be safe to pass it across intents..


Possibly a new personal best for length of the commit message.


one more thing.. calling intent.getExtras().getString() is redundant all throughout this method.. kindly find a way to place it inside a variable and that variable should be used when values are assigned to your respective views..


This revision is clean.


below is the code i use to escape strings. a basic regex to escape ! , \ | and / characters


i am sensing something weird here.. why were you setting the success display result when an NPE occurred?


we use mEscape to escape only for Filter Expressions' values


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.406](http://teamcity.monogame.net/viewLog.html?buildId=9059&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 521, ignored: 6 Build time: 00:19:28
### Failed tests
```
MonoGameTests.exe: MonoGame.Tests.Visual.MiscellaneousTests.Clear: <no details avaliable>

```


No toques mi routes.php!


@calebamiles , sorry, we missed to commit the latest changes, will submit it right now.


should be 7 done


test


OK, I agree the code that sets pipefd in `DragDataCollector::ReadDragData` should move to the UI thread.


Missing jq-console in package.json


> Instead it will be used for its cross-platform IO/threading routines and for string manipulations.

Ahh, that clears things up quite a bit... But standard C++ offers similar facilities?

> However simply replacing some low level routines with their more secure versions (always low level...) would bring nearly nothing.

Well, `lstrcpy` are totally ***not*** cross-platform, so just changing to `wcscpy` (and friends) would improve compatibility...

> Despite all that, the application currently works. Consequently, to achieve what we want, we choose to  freeze  as much as possible the current code base and to rewrite progressively some parts of the application to offer a better architecture (and not simply better routine calls....).

Yeah, freezing the code base ensures that the behavior won't change, but is it *really* safer to keep the good parts of the code locked in the same room as the insane/psychopathic/murderous parts, than it is to replace the insane/psychopathic/murderous parts with their equally competent, but more stable relatives?


This condition is not correct. It should be `!ok && !g.AllVisible`, but perhaps it is easier to make it
```go
if ok || g.AllVisible {
    nodes = append(...)
}
```


Does this look about right?

I'm concerned that this seems to put a new Python object on the heap for each character.

```C++
+084:         for i, ch in enumerate(string):
+085:             if ch.isspace() != in_ws:
    __pyx_t_2 = __Pyx_PyObject_GetAttrStr(__pyx_v_ch, __pyx_n_s_isspace); if (unlikely(!__pyx_t_2)) {__pyx_filename = __pyx_f[0]; __pyx_lineno = 85; __pyx_clineno = __LINE__; goto __pyx_L1_error;}
    __Pyx_GOTREF(__pyx_t_2);
    __pyx_t_13 = NULL;
    if (CYTHON_COMPILING_IN_CPYTHON && likely(PyMethod_Check(__pyx_t_2))) {
      __pyx_t_13 = PyMethod_GET_SELF(__pyx_t_2);
      if (likely(__pyx_t_13)) {
        PyObject* function = PyMethod_GET_FUNCTION(__pyx_t_2);
        __Pyx_INCREF(__pyx_t_13);
        __Pyx_INCREF(function);
        __Pyx_DECREF_SET(__pyx_t_2, function);
      }
    }
    if (__pyx_t_13) {
      __pyx_t_3 = __Pyx_PyObject_CallOneArg(__pyx_t_2, __pyx_t_13); if (unlikely(!__pyx_t_3)) {__pyx_filename = __pyx_f[0]; __pyx_lineno = 85; __pyx_clineno = __LINE__; goto __pyx_L1_error;}
      __Pyx_DECREF(__pyx_t_13); __pyx_t_13 = 0;
    } else {
      __pyx_t_3 = __Pyx_PyObject_CallNoArg(__pyx_t_2); if (unlikely(!__pyx_t_3)) {__pyx_filename = __pyx_f[0]; __pyx_lineno = 85; __pyx_clineno = __LINE__; goto __pyx_L1_error;}
    }
    __Pyx_GOTREF(__pyx_t_3);
    __Pyx_DECREF(__pyx_t_2); __pyx_t_2 = 0;
    __pyx_t_2 = __Pyx_PyBool_FromLong(__pyx_v_in_ws); if (unlikely(!__pyx_t_2)) {__pyx_filename = __pyx_f[0]; __pyx_lineno = 85; __pyx_clineno = __LINE__; goto __pyx_L1_error;}
    __Pyx_GOTREF(__pyx_t_2);
    __pyx_t_13 = PyObject_RichCompare(__pyx_t_3, __pyx_t_2, Py_NE); __Pyx_XGOTREF(__pyx_t_13); if (unlikely(!__pyx_t_13)) {__pyx_filename = __pyx_f[0]; __pyx_lineno = 85; __pyx_clineno = __LINE__; goto __pyx_L1_error;}
    __Pyx_DECREF(__pyx_t_3); __pyx_t_3 = 0;
    __Pyx_DECREF(__pyx_t_2); __pyx_t_2 = 0;
    __pyx_t_4 = __Pyx_PyObject_IsTrue(__pyx_t_13); if (unlikely(__pyx_t_4 < 0)) {__pyx_filename = __pyx_f[0]; __pyx_lineno = 85; __pyx_clineno = __LINE__; goto __pyx_L1_error;}
    __Pyx_DECREF(__pyx_t_13); __pyx_t_13 = 0;
    if (__pyx_t_4) {
```


For consistency's sake, Log calls should be predicated on the value of Constants.LOGGING.


As above: for consistency's sake, Log calls should be predicated on the value of Constants.LOGGING.


Cool, thanks for changing this.


bak


Thanks.


This test was provided by @stephentoub


@keithbostic, sorry about this: I understand the intent and we talked about it yesterday, but on review this doesn't look safe to me.

Updating `btree->flags` without any protection seems fragile.  I don't know of any specific bug today, but it isn't hard to imagine some other flag change getting overwritten due to this if an application is opening and closing lots of session handles.  We mostly get here through `open_session`, which is holding `conn->api_lock`, but I don't want that to be the only thing protecting `btree->flags`.

We could just set `cache_resident=true` and `log.enabled=false` when creating the metadata, but that doesn't help for existing databases.  That would be my preferred approach for the lookaside table (including for any new flags it needs).

We could get the tree in exclusive mode here, which would make this safe, but then we'd have to deal with `EBUSY` failures if the metadata is already open in another session.

FWIW, I think we can just assert `!F_ISSET(btree, WT_BTREE_NO_LOGGING)` -- I don't see any way that could happen unless someone hand-edited the turtle file.

Lastly, we could do something like this:

```c
if (!F_ISSET(btree, WT_BTREE_IN_MEMORY) || !F_ISSET(btree, WT_BTREE_NO_EVICTION))
    WT_WITH_SCHEMA_LOCK(session,
 	F_SET(btree, WT_BTREE_IN_MEMORY | WT_BTREE_NO_EVICTION))
```

I don't much like it, but if you really don't want the special case code where we are configuring btree handles, that's the best approach I can think of.


What if the original string ends with a backslash - wouldn't escaped string end with an escChar?


＋抗体検査で『2.0(-)未満』のような記載方法があったため、検査結果のパターンを編集した


2-category or bicategory?
http://ncatlab.org/nlab/show/bicategory

The latter is a weak form, while 2-category is strict.  I guess we need to try 2-Cat and see?


Don't block on this, but this Interop.cs file is getting fairly long and contains interop for both Windows and Unix... we should a) at least split them apart into Interop.Windows.cs, Interop.Unix.cs, etc., and b) reconsider putting all of this stuff into the normal src\Common\src\Interop folders and using the definitions from there.


seems legit


What even is feature consistancy?


I'm not a fan of one-liner methods, and while that's a neat hack to chain `ConditionalGet` into the middleware pipeline, it requires far too much mental effort to work out exactly what's happening.  I'd much prefer a more "obvious" implementation, even though it will be at the expense of brevity.


i see this function replicated three times in this change - can these be combined into one?


Whoa!  That has to be a `304` response, *not* a `200`.  Returning a `200` and an empty body is going to end in tears.


Is this a race condition?  What happens if the process exits between the check for HasExited and the time we call Kill?


nice dude


Whitespace around operators, please.  (You do this in a number of places, I won't note each one).


Well you should of known, since the associated weapon no longer got minicrits

I think the only Cond that gives minicrits now is the soldier buff, there's an unkown cond that's intended to be minicrit on kill(Cleaners carbine) but it does nothing


Looks good and I like the idea of a report! We don't always want to write to disk though. Will you add a switch [switch] to make the report generation optional and also allow the user to provide a path for report output? The user may not have access to write to the root of the drive.  I'd be happy to merge then. Good stuff and we appreciate the enhancements!


That's not a good way to test that.  `!headers.has_key?('Last-Modified')` would be a much better way.


I don't quite follow your suggestion here. I don't see how to make it safe to post the OnDragDataReceived message directly to the FILE_USER_BLOCKING thread.


GJ


Is there a reason that you're not using Sass for this file? It would probably improve maintenance and readability, because you could define variables and mixins to reducre repetition.


Since this is a continuation of the previous line, it should be indented (or just kept all on the same line; this codebase isn't a stickler for 80 column line width).


Yes, I was thinking of an enhancement with a switch and the report filename
input. I'll get that in and then send over the updates.

On Tuesday, July 21, 2015, rvrsh3ll <notifications@github.com> wrote:

> Looks good and I like the idea of a report! We don't always want to write
> to disk though. Will you add a switch [switch] to make the report
> generation optional and also allow the user to provide a path for report
> output? The user may not have access to write to the root of the drive. I'd
> be happy to merge then. Good stuff and we appreciate the enhancements!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ChrisTruncer/Egress-Assess/commit/3bdbbeaa3b8885df3450446f1306c8ecf4e7660b#commitcomment-12288941>
> .
>


bak


Is this copyright notice still accurate? Looks like you've modified it...


I haven't counted, but it looks like this should all fit onto one line without straining things too much.


Whitespace around rockets, please.


I think it would be best to adopt a scheme here, to regroup navigation features. I'm thinking something like `INavigateAllImplementations` - "find symbol" would then be `INavigateFindSymbol`, and we can have a `INavigateAllReferences` and later, `INavigateRegexSearch`


In this repository you will find two files:
- run_analysis.R, which is R-code that treats the data creating a tidy data-file
- Read.Me, which has comments about what this R-code does


A simpler implementation would be:

```
(defmacro if-not (condition if-true &optional if-false)
  `(if (not ,condition)
       ,if-true
       ,if-false))
```


Changes updated


Good work!  Just keep in mind you are only checking on name, not any other property.  That is fine, as long as it is intentional.  Generally if you are only checking one property you are better off using something more unique like e-mail


This seems like an odd function name, and I'm very wary of a function that has so many arguments. It looks to me like you're only using a few of the arguments, so you can probably get rid of the rest. If not, then maybe put them in a JS object literal so they can act like named parameters?

Or is this function signature mandated by the Sugar environment?


You might consider using `forEach`, which has been around since ECMAScript 5.


This one's closer to a suitable use for a one-liner method, but still... it just feels wrong.


앗! 그러네!


Hey I was wondering if you could help integrating this into the woocommerce store


hpwh_residential (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1750 of 1757 tests passed)

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


Comma placement is wonky there.


Yes, minitest could really do with a `let`-style operator, couldn't it...


`null` is falsy in JavaScript, so you can probably just do `if (startBlock)` and `if (note)`. Also, I think you're missing a space after `if` here. :)


btw, I tried opening the Sonata account, but I think the process is not DIY, or maybe I couldn't find the right registration page. All I could put was my contact info.


But slime indents it correctly using the &body tag, that was the real reason.


This looks much cleaner but is more inefficient since the switch is evaluated for every client.


I don't see the value in using a global for this.  It just seems like it'll confuse the situation.


this should use the `computed` syntax instead of prototype extensions.

```
style: Ember.computed('name', function() {
     return 'color: ' + this.get('name') + ';';
   })
```


Would "global" be a better description than "general"?


标点符号没有统一使用中文标点，s/:/：/


`s/ressource/resource/`


这里在 [1] 之前也追加一个 ：吧，跟下面格式保持一致。


Could we use a higher resolution version of the badge? On Retina the edges don't appear as sharp as they could be.


s/目前/当前


Again, you don't need the `!= null`, and you might get clearer code from reversing the conditions:
```js
if (infoDict && typeof(infodict) === 'object') {
```
Also, remember never to use `==` in JavaScript unless you fully understand its semantics. Generally you should always use `===`. (It probably won't make much difference in this case, but it's a good habit to get into.)


Missing a `)` before `;`.


This gets a little deep into jargon and processes the user may not be comfortable with yet (aliases, factory, create method).

How about something along the lines of
> The `subject()` function lets us create a new instance of the component along with any initial values we want it to have.


Not sure a `.tar.xz` is the most friendly file format for Windows users. Do we have a `.zip` available?


遗留的问题？


I am not sure we can use Chrome Script here since it's not the same as Firefox addon.


The user likely has no idea that `.set` is not instantaneous.  Can we explain this better?


标题翻译跟上面一致吧。


Where is the `tab` variable?


Why not put the `var` declaration before the conditional? That way you don't have to repeat it.
You also might be able to make this whole thing more concise, perhaps something like:
```js
var turtleName = note ? (note+'note') : '';
turtleName += i.toString();
```


Do we know why this was causing a problem?

If it's part of the original codebase we'll have to put it back (at some point) so that there's a better chance that's merge back in.


s/提高/改善/


This is amazing.


nni


i have made $config['base_url'] to null and $config['index_page'] also. An anchor tag i have done it like
<a href="<?php echo base_url(); ?>admin/index" >Home. but its not taking me to the desired page. I'm using codeigniter 3. In htaccesss i have put rewriteengine on.. etc like in this project. but still i'm not redirected to the desired page.. Can you help on that.. im very new codeigniter.


two times `setPos` on `m2`


Давай поле оставим строкой, а здесь будем вытаскивать ее при помощи Hex().


Похоже на правду :)


hmm sure ?


Тут по-хорошему нужно создать и наполнить ProblemDetails, а потом добавить их к списку Details (поле у проблемы).


To avoid doing twice the same test you could do something like this:

    $method = "POST";
    if ("PATCH" === $request->getMethod()) {
        $method = "PATCH";
        $option["validation_groups"] = false;
    }
    $option = array('action' => $action,'method' => $method);


add to doc comment of `postTransform`


Only took us three years to add this… :)


remove `DEFERRED`?


very nice :)


Causes IV in fullscreen to get a white space at bottom.


P.S. The dumping-loading process now REQUIRE ruby2.1.6 to preserve the structure of the output variable "dyn_segs"!
Otherwise a "<no_method> error" is raised!


This closes #1.


rename that method later :)


Hey, I did not realize (or expect) my work-in-progress is already merged with the stable branch :laughing:
OK, I'll soon open another PR with small fixes, rather than new stuff with more small bugs


Ah I see. I'd typed ch as unicode. It's unintuitive to me that I should type it as Py_UCS4 (a struct, right?) and then write ch.isspace().

Now I get:

```c++
+091:         for uni_chr in string:
  if (unlikely(__pyx_v_string == Py_None)) {
    PyErr_SetString(PyExc_TypeError, "'NoneType' is not iterable");
    {__pyx_filename = __pyx_f[0]; __pyx_lineno = 91; __pyx_clineno = __LINE__; goto __pyx_L1_error;}
  }
  __Pyx_INCREF(__pyx_v_string);
  __pyx_t_6 = __pyx_v_string;
  __pyx_t_10 = __Pyx_init_unicode_iteration(__pyx_t_6, (&__pyx_t_7), (&__pyx_t_8), (&__pyx_t_9)); if (unlikely(__pyx_t_10 == -1)) {__pyx_filename = __pyx_f[0]; __pyx_lineno = 91; __pyx_clineno = __LINE__; goto __pyx_L1_error;}
  for (__pyx_t_11 = 0; __pyx_t_11 < __pyx_t_7; __pyx_t_11++) {
    __pyx_t_1 = __pyx_t_11;
    __pyx_v_uni_chr = __Pyx_PyUnicode_READ(__pyx_t_9, __pyx_t_8, __pyx_t_1);
+092:             in_ws = uni_chr.isspace()
    __pyx_t_4 = Py_UNICODE_ISSPACE(__pyx_v_uni_chr);
    __pyx_v_in_ws = __pyx_t_4;
  }
  __Pyx_DECREF(__pyx_t_6); __pyx_t_6 = 0;


Please use two space indents as per rest of source


Thanks @nao-pon. By the way, I also successfully integrated Ace (I also tried your Ace integration but I got some console errors - I'll check that later) and I found an easier way to auto-detect the file mode, without having to specify all those extensions.

I'm using the [already built Ace package](https://github.com/ajaxorg/ace-builds/tree/master/src-min).
Just add the following to `<head>` (you can remove the individual modes, eg. `mode-php.js`, `mode-css.js`, etc.):

```html
<script src="path/to/Ace/ext-modelist.js"></script> <!-- include modelist script -->
```

And then, in the `editors` section:

```php
load : function(textarea) {
    $(textarea).hide();
    $("<div id='ace-editor'></div>").insertAfter(textarea).text(textarea.value);
    aceEditor = ace.edit("ace-editor");

    modelist = require("ace/ext/modelist"); // tell Ace to load modelist extension
    detectMode = modelist.getModeForPath(this.file.name).mode; // ask Ace to get mode according to filename

    aceEditor.setOptions({
        mode: detectMode, // set the mode option
        theme : "ace/theme/tomorrow_night_eighties",
        fontSize: 14,
        ...
    }),
},
close : function(textarea, instance) {
    aceEditor.destroy();
    $(textarea).show();
},
save : function(textarea, instance) {
    textarea.value = aceEditor.getValue();
},
focus : function(textarea, instance) {
    aceEditor.focus();
}
```


also very nice


ReplaceCode with this one
updateMonth: function (e) {
            var month = parseInt($(e.target).val());
            var isLeft = $(e.target).closest('.calendar').hasClass('left');

            if (isLeft) {
                this.leftCalendar.month.month(month);
            } else {
                this.rightCalendar.month.month(month);
            }
            this.updateCalendars();
        },


Please use 2 space indents


updateMonth: function (e) {
            var month = parseInt($(e.target).val());
            var isLeft = $(e.target).closest('.calendar').hasClass('left');

            if (isLeft) {
                this.leftCalendar.month.month(month);
            } else {
                this.rightCalendar.month.month(month);
            }
            this.updateCalendars();
        },


Oh, so sorry, I didn’t know… I won’t change it in the future.


no need for toString.


 updateYear: function (e) {
            var year = parseInt($(e.target).val());
            var isLeft = $(e.target).closest('.calendar').hasClass('left');

            if (isLeft) {
                this.leftCalendar.month.year(year);
            } else {
                this.rightCalendar.month.year(year);
            }

            this.updateCalendars();
        },


No problem. You couldn't know. I'll add it somewhere.


Change to XFUtil.isNull


This is only  internal hack for using SSL in the HTTP connection. Don't use it. So, I can be to use set_hook.


Это старый код.


ah, ok, interesting, wasn't aware of that ...


I'd rather use the 1.8.3-SNAPSHOT version instead


3dp? и ресурсы


when there's a single use, you should inline it


What about "must be compatible with PHP version declared in composer.json"?


Hi,
You should point hue to the prestogres pgpool port. That should be the port
in prestogres.conf.  What error message does hue output? Do also check the
hue logs for more robust output.

The file names were updated a few months ago as the daemons were upgraded
too, I've yet to go back and update the blog with that ;-)
On 22 Jul 2015 04:41, "liusztc09" <notifications@github.com> wrote:

> Hi @Ile2 <https://github.com/Ile2>,
>
> I was following your log post(
> https://medium.com/@ilkkaturunen/integrating-presto-with-hue-61702b244839)
> to set up Presto with Hue. I succeeded in installing prestogre and run in
> shell, but fail to configure hue to pick it up. I'm running it with AWS EMR
> on the master node.
> 1. I could not find the pgpool.conf and pool_hba.conf, instead I modified
> prestogres.conf and prestogres_hba.conf. Also, I modified hue.ini instead
> of pseudo-distributed.ini and here are my parameters:
> [librdbms]
> # The RDBMS app can have any number of databases configured in the
> databases
> # section. A database is known by its section name
> # (IE sqlite, mysql, psql, and oracle in the list below).
>
> [[databases]]
> [[[postgresql]]]
> nice_name=”Presto”
> name=default
> engine=postgresql
> host=127.0.0.1
> port=5439
> user=pg
> password=
>
> What I'm not sure is the port, which one should I use? Postgresql server
> runs on 5432 while pgtool listens 5439. I tried both but neither worked.
> Thanks you in advance for any advice. This might not be the right place to
> ask the question. Sorry about that!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ile2/hue/commit/6d02db1f0b58b4103bc131526f4734b4a9a5a2d5#commitcomment-12290176>
> .
>


guava version should belong to top-level pom.


top-level, too


hi


хз, он не всегда срабатывает, но попробую


Ah, I hear you. Thanks for clarifying.


too many cassandra


All Telugu Songs Can be downloaded at http://www.superhitsongs.in/


ок


omg :+1:


replace the code to fix drop down selection issue


I have to remove `.encode("utf-8")` for this to work (Python 3.4 with IPython 3.2.1)


Yes, I would say so. But this was a temporary version, the current one has a few guidelines (and I’ll add those two you suggested). :smiley:


Add javadoc for new parameter


This feature already exists in all tuttle writers:
https://github.com/tuttleofx/TuttleOFX/blob/develop/libraries/tuttle/src/tuttle/plugin/context/WriterPluginFactory.hpp#L51


konstruktor kopiujacy - zeby tutaj `pudlo` nie wskazywalo na ten sam adres co stare (dlatego po dodaniu 11 wczesniej bylo dodawane do obu paczek) - patrz linia `11` gdzie konstruktor kopiujacy z Towar kopiuje kazdy element z listy


I think you can do this:

    title = HiddenField(..., widget=TypeaheadWidget())


ask for an immutable list


Hello,
I don't understand the use of a temporary table. It is not working for us, and we needed to use a previous version of the addons to work.
If you create a temporary table, you do it for everyone, so you have some risk that several people using the filter in the same time just erase the table each other, and the results can't be good for everyone ?


Rekursywnie wywolywany konstruktor kopiujacy - kazdy element w tej linked list bedzie kopia (i dlatego tez w destruktorze musimy je wsystkie po kolei pousuwac tez rekursywnie


same comment here as before


Zamiast usuwac samego siebie, usuwamy nastepny element w liscie rekursywnie


@ascriptmaster, do you test your client code? :|


final


immutableList


We have more than one blog?
:water_buffalo:


谢谢，请提供blog地址看下样式


are you sure this .stream is necessary ?


CassandraModule::moduleTypes


you should collectAndThen to create the immutablemap from the mutablemap


The root issue here is that this inplementation binds to a port < 1024 which is a problem because we want to run our test suite without sudo.
By telling the socket to bind to port 0, it will bind to a pseudo random port above 1024.
For examples on how we did this in the other implementation check the test cases for bacnet that I refactored a week ago or so.


I don't understand why you use both of these
```html
<script src="path/to/Ace/ext-modelist.js"></script> <!-- include modelist script -->
```
```javascript
modelist = require("ace/ext/modelist"); // tell Ace to load modelist extension
```
isn't doing the same thing?


Hey!

C'est une integration avec l'outil logrotate?


Ah thanks. Okay I'll hold off on any releases until those things settle.


@dg Good idea, updated.


garylog :question:


Created https://github.com/mysociety/alaveteli/issues/2675, https://github.com/mysociety/alaveteli/issues/2676


Argh. Cheers.


RegisterReceveiver -> really?

RegisterReceiver


will it work without this deps?


This change is *more* expensive, because it unconstitutionally calls connection.getTransactionIsolation().


@chingor13 , where I can find test for it ?


No problem. But yeah, https://github.com/chrilith/scummvm/commit/4887796888772d1347846ee58227d013afa0e19a is a case which is not working in C :-).


일단은 모델에는 카운트를 추가하지 않았으면 좋겠습니다.
count() 함수를 쓰면 보다 안정적으로 대체가 가능하고 성능 문제는 지금 단계에서는 측정하기 어려운 면이 있는 것 같습니다.


Ja, war Unwissen, keine Absicht. Kannst du das korrigieren?

Am 22. Juli 2015 10:20:20 MESZ, schrieb Flow86 <notifications@github.com>:
>wär sinnvoller gewesen einen pull+rebase zu machen, dann gibts keine
>komischen "merged XXX into ..." commits
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/Return-To-The-Roots/s25client/commit/c02523716a96ef0535255259a2b03ac5d402a58a#commitcomment-12293020


@lingxiaoyang Is "coarse" and "fine" something you use throughout the client? I find it weird that "coarse" means "fetch all" but "fine" means "fetch one." If it's not something that's throughout the client (i.e., if it's a not a big change) I would prefer to see "fetchRunjobs" and "fetchRunjob".


Yup, same bug ;) here


The only test I can think of is checking that two equal Certificate
instances produce matching hashes. Given enough test data, you could test
that it produces something close to a uniform distribution of hashes, but
that's not really practical in this case.

On Wed, 22 Jul 2015 at 10:32 Paul Hadfield <notifications@github.com> wrote:

> Is there any test involving adding Certificate instances to a set that
> would demonstrate that *hash* does what it should do?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RJPercival/certificate-transparency/commit/5c5be65e28384d86f4ea6df3029e559719c68664#commitcomment-12294191>
> .
>
--
Google UK Ltd | 123 Buckingham Palace Road | London SW1W 9SH | Registered
in England Number: 3977902


I'm wondering whether we should add a company name to this directory, i.e. include/lib/cpus/nvidia/aarch64/denver.h, and move the ARM ones to include/lib/cpus/arm/. However, there are not too many yet so we can leave it as-is for now and move later if needed.


This is in the `job` infrastructure (`jobs/core.py`), but it seems like it's actually part of the rodan application itself. Why did you put it here?


why is this necessary?


Mitt forslag til namespace: cl for catalinker


>>>>> "ZHONX" == ZHONX  <notifications@github.com> writes:

    ZHONX> Hello, I have compiled your wine stlink but I don't understand
    ZHONX> how to use it.  I have successfully installed st-link utility.exe
    ZHONX> and I have added rule.d for my Jtag in linux(I use openOCD on
    ZHONX> linux and work well).  In wine, st-link utility don't find my
    ZHONX> st-link v2, how to link USB peripheral (other than storage mass)
    ZHONX> from Linux to wine?  Thanks lot of for your contribution.

I deleted and recreated my stlink branch on githup in the meantime. A
Readme.stlink was added and I tested rebuilding.

Bye
--
Uwe Bonnes                bon@elektron.ikp.physik.tu-darmstadt.de

Institut fuer Kernphysik  Schlossgartenstrasse 9  64289 Darmstadt
--------- Tel. 06151 162516 -------- Fax. 06151 164321 ----------


@hrydgard guess you forgot to merge native changes ~

    fatal: reference is not a tree: 477568b3485e9921ede59c26b266c762b1fbd066
    Unable to checkout '477568b3485e9921ede59c26b266c762b1fbd066' in submodule path 'native'


@CauehCraft  I would be happy to but I'm VERY limited on knowledge. I only know how to create items and render blocks. Right now I'm trying to learn how to add a crafting recipe. But I can tell you where to start! Go on zhuoweizhang.net/addongen


I can still help you but only with what I know... Sorry :(


Hi @mjalvarez, this put back our postman structure to this - non-folder version

![Stellar API postman](http://i.imgur.com/7O8ueWh.png)


锴哥，我觉得这里的获取文件的路径得换下，这个文件应该是要放到 hadoop 的文件夹下面去。应该像下面这样获取文件路径:
 private static final File TEST_DIR = PathUtils.getTestDir(TestMD5FileUtils.class);
  private static final File TEST_FILE = new File(TEST_DIR,
      "testMd5File.dat");


Good point. Just pushed a new version with generalization


Ohhhh i change , Thanks Ysurac


A short explanation what this implies would be cool :smiley:
Or does it just clean up mechanics without touching roules?


merge


And bunch of requests got squashed :grin:

I reverted it back to ac75416ec058147ffda67de1328b805bcc4c3a4a for now. I think we already have ~80 requests. This is only 58.


Thanks! :)


stabby lambda it is


Use REQUEST_TIME instead of time().


One reason for 10000 vs 100 was that it enforces also a minimum number of iterations no matter how slow the device is or how small the compute time was set to (eg 1 ms).

On the other hand, the library leaves open plenty of other options for choosing bad crypto parameters already (you could use a 32-bit Blowfish key if you liked) and without the ratio test or something like it we have no way to effectively test timed PBKDF2, at least not without mocking the time handling.

Perhaps just clearly documenting that an application wants to set at least 100 ms in practice and moving to 100 round multiples is sufficient. Another would be to allow specifying both a (minimum) iteration count and a timeout to PBKDF, and it iterates until both have been satisfied.


Did the line `(require 'bbdb-loaddefs)` work after your mu4e setup, i.e. after the line `(require 'mu4e)`?


Also made indentation as we said - 2 spaces


I think he just wanted to let you know (as always) that R2 was much faster in implementing this feature than CMaNGOS. Nice that this is back, it has indeed some traditional values.


I wouldn't put laravel, it is deprecated.


Just for your info: I've recently made changes to the master-branch of csharp-mode to make it handle compilation through devenv.exe, which I'm guessing you are doing here.

Can you test if these changes work for you or if you still have need to maintain your own changes? It would really be appreciated.


The `wpt_event_time` filter was removed but might still be in use in some installs.


All right. Are you plan to support symfony?


Except for the laravel thing, LGTM


Tanks Uwe Bonnes,
I retried to build with your new branch.
Your help is precious


I think you're right.
We had a leak here and something had to be done; the fix however wasn't good and later it was changed.
Wish I knew the PR ...


python is completly removed?
i see in Make file: "/ -I /usr/include/python2.7/"


No. Not really. Symfony style is much better supported by PHP-CS-Fixer


https://github.com/Zarel/Pokemon-Showdown-Client/pull/426#issuecomment-123429748

> Should be fine now. Wi-Fi situation is still a problem for me so I haven't had the opportunity to client test, though.

Blame @Zarel for merging even though I said I hadn't client tested yet...


Seems like this commit broke [CI](https://travis-ci.org/jquery/css-chassis/builds).


First line is the actual resource path, like you include another script or stylesheet. elFinder needs to know this.

Second line is actually attributing the modelist extension to the variable `modelist`. It could be another variable name like `myownvarname = require("ace/ext/modelist")`.

You can even turn it into a one liner like `detectMode = require("ace/ext/modelist").getModeForPath(this.file.name).mode`.


Nah, pff :smile:

Find it :see_no_evil:


@Jan-E See also the [discussion in the PR](https://github.com/php/php-src/pull/1429).


Of course you could do that. But that would weaken security for performance.
It's a matter of your priority.


Well that's "new"...


Fixed in b33246c


Why?


We could easily fix this afterwards :) but yes, something for sure to keep in the guidelines for eventual future translations!


Fixed in 54ac1f0, 3bb3c2a


Yeah, short_name is equal to candidate_id... I believe it's named short_name because that is how flask sends it's form elements


Will fix :)


My understanding is that it would go as a `#define` but instead be defined in the `Style` class.

https://github.com/rism-ch/verovio/blob/master/src/style.cpp

Then you're not relying on (limited) preprocessor math.


Oh, thanks @hemanth.


this change is reducing two calls to one.
the condition before was ......if (transactionIsolation != connection.getTransactionIsolation())


KO Myung-Hun dixit:

>If not, PATH should be quoted. check.t, too.

Indeed, thanks for spotting this!

bye,
//mirabilos
--
“It is inappropriate to require that a time represented as
 seconds since the Epoch precisely represent the number of
 seconds between the referenced time and the Epoch.”
	-- IEEE Std 1003.1b-1993 (POSIX) Section B.2.2.2


One thing I need to do is combine the filter and sorted into a single lambda so there aren't two iterations... I will get that done today


After a further look, it looks like minitest DOES have `let`. Not sure what made me think the contrary. Maybe because it is not mentioned in the `README`. I'll use it then.


I think combined with perfect forward secrecy aes 128 should be more than enough.

But ok.


@port is optional and defaults to 8086


Oops, thanks. I'll fix it tonight.


lol not sure, I literally spent hours trying to figure out how to query inside the test, if you can direct me to where the function is I will use it :+1:  :)


Could we define a variable for `($gutter / 6)`?


It is ordering by timestamp.. I could create a new method in the Vote class if that would be better than using lambda filter sorted once I get the two functions into one lambda today


I am open for any suggestions :)


MIT :O

ça vaut pas un beerware...


@tijmenb locally or in `govuk_frontend_toolkit`?


Please remember not to use spaces around "=".


I may be missing something, but it seems to me that, if fopen() succeeds but flock() fails, you are not fclose'ing the file.
In the code that I had proposed in my pull request I did close the file in both cases. Back then I thought you had cleaned up my (undeniably messy) code and made it more compact, but I think you're missing that case.

I guess it's almost harmless  though, as it will only "leak" the resource of the opened file (readonly and with no lock) for the duration the request, at the end of which it will implicitly be closed anyway.


You can just use `for idx, element in enumerate(elements)` here.


Non ca créé juste le fichier dans /etc/logrotate.d pour que le logrotate système soit actif sur les logs de pm2. J'hésitais à le mettre dans le `init` mais ca marchera pas sur os x ou windows par ex.


The AES test is actually testing DES?


To be honest. I didn't dig that deep into this crypto stuff. I searched around the web, found stuff, read stuff, put stuff together. Ciphers (with order) are a recommendation from another admin and SSL Labs give me a good rating for my settings.
I'm perfectly fine with changing the cipher order. But I currently don't want to invest time to test that change (read stuff, rollout with changes and run tests again, ...).

Feel free to change it (if possible), close it, do anything with it. This is what I did and it looks fine to me. It's "good enough" and I wanted to share it.
Of course it's possible, that I will be changing my own setup/settings, if I get (performance) problems with it.


1: M111 unrecognized component name: utils


Is it not possible to include all files in a directory? That's how reicast is designed right now -- most of the cases even if you include the code there's #ifdef guards to check if the "module" is enabled or not.


You're right. There is a spelling mistake in the method. See  xbmc/xbmc/messaging/ApplicationMessenger.h

```
void RegisterReceveiver(IMessageTarget* target);
```
Sorry also in the comments is a mistake.


Yes :)
just needed to push everything from a machine that I am going to discard...


Yes it is possible (and it's not too difficult AFAIK). I just felt lazy but I could/will add this.


> unfortunately the mapback model is not enough to test

Do you have any idea to have a nice test to check if the behind logics work or not?


commit


It's fine.
would you please ask the author to add a tag for version 1.0.2?
thanks:)


You can also use
```
Contact.getFile req.params.contactid, picture
```

To avoid 1 request.


if you don't have any parameters, leave the brackets away
constructor: ->


no need to add _attachments with cozydb


I updated some strings in the thrown exceptions to be valid English.


just use:
if value?

this tests the variable for null or undefined


@zidein07
```js
window.addEventListener('keydown', function(e) {
  console.log(e);
});
```


Not sure, but since you cleaned it up so nicely in https://github.com/alphagov/collections/commit/2d48adb46a79cb189e12568dc753650940cb327b it seemed logical to use something like `$gutter-one-sixth`.


Just test ki


I benchmarked "if condition" vs. "new[] { }.Contains". Second construction is far less efficient. Even with declaring array as static variable.


I suspect there is an error here as a result of some copy/paste. `Store` object doesn't `reviewed_on` and `reviewed_by` properties. Let's do s/self/unit/.


Why do you need this check? Seems redundant and from what I understand Maven version range syntax does not require range to start with either of these characters (although I agree that in most practical cases this will be the case).


let's lowercase these `success` "returns" so that the cli/http wrapper do not have to parse uppercase.


Please modify this to check against $request_message and to support /icametofarm@SLEnlFarmBot
e.g. ($request_message == '/help' || $request_message == '/help@SLEnlFarmBot')


sss


indentation fixed


what does
mut
do?


```go
srv, err := loaders.LoadServiceDefinition(do.Name, do.Operations.ContainerNumber)
```


Yep, I supposed their is only one root package in a model.

I rewrote  it for multiple roots in f972a3c3ebe31cf346954807b6044ed1438b4cae


comments I dont really care but fixed in https://github.com/xbmc/xbmc/pull/7584 Ill update comments


That's right, although the scope is the same as the previous code: the error already existed (no one noticed? :no_good:)


Yes, both on Travis in in my local tests. I wander if the cast is needed at all, I will see if I can remove it later.


It did on my local machine. :smile:


haha, adding lots of `;` but missing the one in the new line ^^


Atom.


What are your concerns about this change? Apart from the new intra-module dependency, which I think is okay, the change seems consistent with the original MNG-5840 approach, just does `!version.equals( parent.getVersion() )` check more thoroughly. Am I missing something?


How about something along the lines of `image/*`?

Because `bmp`, `tiff`, `cr2` and the likes are also all valid image formats...


where are these thing defined


Yikes, I tried importing it again, and I see 89 requests! Weird how I got 58 requests from importing on a recent `git pull`. And there's the leaderboards! Sorry about that @mjalvarez


doesnt assert need @


Ah. I see... Happy to revert!


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


```go
logger.Infof("%s is not running. Starting now. Waiting for %s to become available", strings.ToUpper(do.Name))
```


hello world


What if action is not into the Snackbar? I guess it will throw exception!


Aww mah gawd mah eyes are BURNING. It started like this?!


...and here we go again.

On Wednesday, July 22, 2015, LolThatSecurity <notifications@github.com>
wrote:

> Aww mah gawd mah eyes are BURNING. It started like this?!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Bukkit/Bukkit/commit/4e8311a6551e8d7794cff73c57a481251b47459c#commitcomment-12295837>
> .
>

--
-------------------
Josh Morgan - Java Developer
Ph: 0488 178 527
Code: github.com/joshmorgan


Noted.


I thought tasks were preferred to Commands?


```go
} else {
  logger.Infof("%s is running.", strings.ToUpper(do.Name))
}
```


A collection of tips & tricks for writing in Emacs: Don't. Use an IDE.


Talk *at* one of our next JSBelgrade Meetups.


Shouldn't this be based on the `kernel.debug` parameter?


@Dolu1990 commit number `0xFF` :notes: :sparkles:


Is it possible to set the style though styles.xml?


Looks correct from first glance. I'm just going to merge this.


Warning! `README.md` file permissions have been changed!


amxdbg.c seh ich zum ersten Mal, kreass wo hastn das her


first commit!!


What do you think of setting the `NSLocalizedDescriptionKey`: https://developer.apple.com/library/mac/documentation/Cocoa/Reference/Foundation/Classes/NSError_Class/#//apple_ref/c/data/NSLocalizedDescriptionKey


In fact it's been broken since 9d69dd11; will fix it in master.


isn't this after window? window uses `xasp`, so this causes the first plot to use defaults, and the second to use config for setting up window.


别注释，直接删掉比较好。


I'm aiming for readability not performance. But I agree that static HashSet is better, what do you say?


And add `[ci skip]` in commit's title, please if you're changing documentation only. It will prevent Travis testing.


Indeed, but in that case, I think that having a setting to control if coverage is enabled rather than tasks that modify a mutable var in the plugin is better.
In its current state, is it impossible to enable coverage **by default** in your project. You'll have to call the `coverage` task to mutate `enabled`.
With this PR, you can add `coverageEnabled := true` to your settings to always enable coverage or use the commands to toggle it when required.


so where are we going here:: constructer; traversal; wrapper; sequencial; parallel


Done in 24bdb358fb9c9cab0cad604f09257bca8a83a270.


:) one more thing please, in ghostdbmysql.h this code:

//
// COHBotDBMySQL
//
class COHBotDBMySQL : public COHBotDB
.
.
.
private:
    static calls initMap() {
	calls ini = {};
	return ini;
   }

this "private"  generate an error at line :
               calls ini = {};
Compiling error: error C2552: 'ini' : non-aggregates cannot be initialized with initializer list
is that code a part of old pyton code? if i comment this "private" I CAN compile in windows!
please help!


For collisions, it might be nice to have a function willFit(coordinates) (or a better name) which takes in a list of coordinates and returns true if they are with in the grid and the grid square isn't taken.

Then you use it to determine if a block can move. So for moveDown it would be some thing like:
var movedCoordinates = block.coordinates.map(function(coordinate) { return [coordinate[0] ,coordinate[1] + 1])
if(willFit(movedCoordinates) {
// Move down
}


does this survo browser implemen web-gl for blender4web or unity


if so which part


Dann doch lieber beim create. Dann gibt es zwar auch Punkte für blosse Klicken, aber das ist egal, weil badges gibt es nur für update. Und da kann ich ja den Level hochsetzen.


Ok


Gut!


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


`name: {{ field.name | tojson }}`


Shouldn't this USER-AGENT be a configurable option too?
I'm currently using a S40 resource, however, if I'd generate a UploadPOST wouldn't WhatAapp be able to see that I'm using a different user-agent and resource?
That could potentially be used to detect purple-whatsapp users which sounds pretty risky to me with all the blocking in the past.


I'd go for `{{ field.id }}-typeahead` (the reason is explained in a previous comment)


Hi @JoramBerger ,

would you mind if i change this log output to debug. It spoils my output(for many pipelines) and i think it is no different to many other settings in which you get invalid results if you select the wrong setting.


Ehh, sorry. Apparently i didn't save the changes in the right file..


It tests positions around the player for solid stuff to parkour on, as I was already adding `dir` to `d->o` there wasn't a real point in doing a directional `collide` call as this could actually test negative if the geometry is especially thin.


Fixed in f4828af


Looks good


باید بعد از for بزاری
  mAdapter.notifyDataSetChanged();


The post_save hook doesn't work on .update method. The message must therefore be sent manually in some way.

On Jul 22, 2015, at 6:35 AM, Andrew Hankinson <notifications@github.com<mailto:notifications@github.com>> wrote:

This is in the job infrastructure (jobs/core.py), but it seems like it's actually part of the rodan application itself. Why did you put it here?

—
Reply to this email directly or view it on GitHub<https://github.com/DDMAL/Rodan/commit/f3a0cbaeca6cbef96e9451db8df5189d64eef01c#commitcomment-12295188>.


Then it should be fixed now :hear_no_evil:


is it possible to merge these changes to master and request for a new release tag


Questions are: does it make sense to order by timestamp or is it used anywhere as such?


It would be, but I only half understand why it's needed or better said why it wasn't before.


추추가...


Not sure, that's why I didn't want to touch it


OK, but why is it defined in `jobs/core.py` when it's part of the "core" Rodan infrastructure (i.e., the code for sending websocket messages is not part of the job execution components, is it?)


Oh, pardon my English. They are for fetching run jobs in a more pinpoint manner.

On Jul 22, 2015, at 6:32 AM, Andrew Hankinson <notifications@github.com<mailto:notifications@github.com>> wrote:

@lingxiaoyang<https://github.com/lingxiaoyang> Is "coarse" and "fine" something you use throughout the client? I find it weird that "coarse" means "fetch all" but "fine" means "fetch one." If it's not something that's throughout the client (i.e., if it's a not a big change) I would prefer to see "fetchRunjobs" and "fetchRunjob".

—
Reply to this email directly or view it on GitHub<https://github.com/DDMAL/Rodan/commit/ddb1d2276eb1f4be245652db71ca3acdaa7a6ee0#commitcomment-12295156>.


Yeah i was thinking about this one, If i make it so that only comment owners can remove their comments despite post ownership it would make things more legit, so im changing the code to fit that need.


I was thinking about: https://github.com/NerdsvilleCEO/elections/commit/ab93ba6b4628e6c25a21b570faaa55cf70aa8ba1#diff-99b7be98a3b0257c41b9b01eafb31221R127


@jensens done


Stray change? (just checking)


Esta parte de aquí creo que no está 100% bien porque unas líneas antes se invoca a serviceContext.deriveDefaultPermissions que modifica los valores de groupPermissions y guestPermissions en serviceContext por lo que no coincidirán con los de ModelPermissions

```java
serviceContext.deriveDefaultPermissions(
 					repositoryId, DLFolderConstants.getClassName());		 					repositoryId, DLFolderConstants.getClassName()
```


Ég myndi frekar nota foreach lykkju frekar en for lykkju.  Bæði er það að það er verið að keyra sizeof() fallið í hvert sinn sem for-lykkjan keyrir (overhead), og svo er bara miklu auðveldara að eiga við foreach lykkjur

```php
foreach($data as $item){
  print $item->title;
}


Yes I am sure. I got a cyclic inference... flatMap waits for a stream, and a list is not a stream.


@Rikk @ariccio Get over it! :)


It's not a question of language, more of style. Rodan (server) uses singular and plural to indicate whether you're fetching one thing, or many things, so I thought that the clients should try to mimic this.


Essa linha não é mais necessária uma vez que o travis.cfg não existe mais.


I would keep the notation for sinogram slicing similar to other reader functions: ``recon_aps_32id_chunk(..., sino=(start, end))``


confused as to why you need the ```eval```. Can you not just type:
```
if system(['pdflatex -halt-on-error ' M_.fname '_TeX_binder.tex'])
    error('TeX-File did not compile.')
end


Ok


No; the http.go caller logs the error so I put the return in to avoid logging it twice.


espace useless :p


on met pas ça en FR ?


@tholden This is really helpful, but could you please separate this out into a new issue. A few additional comments
1. We should not add the doubling algorithm to this function. There is already a separate one: ```disclyap_fast.m```. Algorithm 3 is on purpose different.
2. Where do we ever use sparse matrices in this function? Or did you use this function in https://github.com/DynareTeam/dynare/pull/689
3. For reference, this is also related to #821.


We can try getting the error message and set it to `NSLocalizedDescriptionKey`, yes.


I couldn't seem to get that to work, flask was throwing an error that it was out of context when I ran the test using that, not sure why


eda array_map

```php
array_map(function ($item) {
    echo $item->title;
});
```


Lol le psychopate de la review de commit :p j'avais mais pas fait gaffe que j'avais introduit un espace :D


I would group all 32id (or all) scripts in a single place, and even put them away API reference.


i think this should be an argument of the widget, not the field.


Would be cool to check as it would make things easier :)


maybe better suited as a widget argument? also, missing space after `tojson`


je sais pas ... car potentiellement certains speakers seront anglais donc on aura qu'une bio anglaise... à voir


I agree, but that's the only place to invoke the message publishing code. Two alternatives:

1. Wrap .update method. Could be hard, as the QuerySet object is constructed by Django through .filter method.

2. Add a monitor to PostgreSQL and monitor database changes directly there. Maybe worth to investigate.

On Jul 22, 2015, at 7:39 AM, Andrew Hankinson <notifications@github.com<mailto:notifications@github.com>> wrote:

OK, but why is it defined in jobs/core.py when it's part of the "core" Rodan infrastructure (i.e., the code for sending websocket messages is not part of the job execution components, is it?)

—
Reply to this email directly or view it on GitHub<https://github.com/DDMAL/Rodan/commit/f3a0cbaeca6cbef96e9451db8df5189d64eef01c#commitcomment-12296118>.


Is there any good reason at all to use PBKDF with a time limit rather than a fixed iteration count? I mean, you get different hashing results when you just set a time.


missing `|tojson`? if it works right now it's because *some* dict reprs are valid javascript (if they only contain bytestrings/ints/lists/dicts).


lgtm. Thanks


Hey, thanks for the interest. I think I may have already gotten these bugs, though.

Could you please do a fetch of the most recent version, and see if the problems you have attempted to address are corrected?


@sameersingh The reason this was private is because
```scala
Riedel2010Document extends com.google.protobuf.GeneratedMessageLite
```
leads to a missing dependency error when compiling Riedel2010Example.scala in wolfe-examples. I figured I'd just stop these classes from escaping the scope of wolfe-nlp, but do you have a better suggestion?


By calling ``models.Vote.of_user_on_election(..)``?


I think technically this change gives us call stacks, not only source locations.


why not use an ID? `$('#{{ field.id }}-value')`


either remove the space before `|` or add one after it


I can't build stlinkusbdriver, here is my output, how can I do?
Tanks!

make[1]: entrant dans le répertoire « /home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver »
../../tools/winegcc/winegcc -o stlinkusbdriver.dll.so -B../../tools/winebuild -m32 -fasynchronous-unwind-tables -shared stlinkusbdriver.spec \
  STLinkUSBDriver_main.o -lwine ../../libs/port/libwine_port.a
STLinkUSBDriver_main.o: dans la fonction « submit_wait »:
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:83: référence indéfinie vers « libusb_submit_transfer »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:94: référence indéfinie vers « libusb_handle_events_timeout »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:102: référence indéfinie vers « libusb_cancel_transfer »
STLinkUSBDriver_main.o: dans la fonction « send_recv »:
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:136: référence indéfinie vers « libusb_clear_halt »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:151: référence indéfinie vers « libusb_clear_halt »
STLinkUSBDriver_main.o: dans la fonction « DllMain »:
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:193: référence indéfinie vers « libusb_release_interface »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:194: référence indéfinie vers « libusb_close »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:175: référence indéfinie vers « libusb_init »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:177: référence indéfinie vers « libusb_alloc_transfer »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:178: référence indéfinie vers « libusb_alloc_transfer »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:201: référence indéfinie vers « libusb_exit »
STLinkUSBDriver_main.o: dans la fonction « STLINKUSBDRIVER_STMass_Enum_GetNbDevices »:
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:222: référence indéfinie vers « libusb_get_device_list »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:232: référence indéfinie vers « libusb_get_device_descriptor »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:254: référence indéfinie vers « libusb_release_interface »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:255: référence indéfinie vers « libusb_close »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:266: référence indéfinie vers « libusb_get_device_descriptor »
STLinkUSBDriver_main.o: dans la fonction « STLINKUSBDRIVER_STMass_OpenDevice »:
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:341: référence indéfinie vers « libusb_open »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:345: référence indéfinie vers « libusb_get_configuration »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:351: référence indéfinie vers « libusb_set_configuration »
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:358: référence indéfinie vers « libusb_claim_interface »
STLinkUSBDriver_main.o: dans la fonction « STLINKUSBDRIVER_STMass_CloseDevice »:
/home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver/STLinkUSBDriver_main.c:374: référence indéfinie vers « libusb_close »
collect2: error: ld returned 1 exit status
winegcc: gcc failed
make[1]: *** [stlinkusbdriver.dll.so] Erreur 2
make[1]: quittant le répertoire « /home/zhonx/Bureau/UweBonnes/wine/dlls/stlinkusbdriver »
make: *** [dlls/stlinkusbdriver] Erreur 2


Can we get rid of this method and ``persistColumnValues()``? I'd prefer if we made the user's intention more explicit.


Yes


also, this can be done in a cleaner / more efficient way:

```
    var field = $('#...');
    if (!field.length) { field = $(..);}
    field.val(...)
```


LGTM


"Coarse"   To fetch run jobs of a particular workflow run.
"fine"  to fetch run jobs of a particular workflow run and of a particular origin resource.

The reason was to prevent fetching too many run jobs from a huge workflow run. But we could have a better name, also documentation... I never made comments in the demo client.

On Jul 22, 2015, at 7:42 AM, Andrew Hankinson <notifications@github.com<mailto:notifications@github.com>> wrote:

It's not a question of language, more of style. Rodan (server) uses singular and plural to indicate whether you're fetching one thing, or many things, so I thought that the clients should try to mimic this.

—
Reply to this email directly or view it on GitHub<https://github.com/DDMAL/Rodan/commit/ddb1d2276eb1f4be245652db71ca3acdaa7a6ee0#commitcomment-12296177>.


Yes, perhaps this was just being over-cautious


lol et encore tu bosses pas avec moi :p


I think you should keep the column's original name somewhere in the new name.


My main concern is pulling in a dependency on maven-artifact... but if others are fine with that then I'm ok too


Were you trying to access ``flask.g`` then? If so I understand the error, otherwise not at all :)


Maybe we could go back to the version where we could specify just the measurement, set of tags, set of fields ?
Example:

```php

$tags = array('host'  =>'server01', 'region'=>'us-west');
$fields = array('value' => 0.64);
$point = new Point(
            'cpu_load_short',
            $fields,
            $tags
        );
```

So then we can handle this problem as @skynet points out


Done.


Those seem like pretty extreme options. Can't you just put the `publish_message` method in another file and import it from there? Like a dedicated "websocket.py"?

`from rodan.websocket import publish_message` seems pretty good, doesn't it?


@skmp: Updated with globbing of the source files.


psr2 : bil a milli `if` og sviga. Bil a milli sviga og krullu. Krulla, bil, else bil krulla

```php
if ($a < 10) {
    print $a;
} else {
    print "Stærri en 10";
}
``


Perhaps we could come up with a better class name. What do you think about ``Entity`` as in JPA?


This was really for #40.


What is the use of the `id` field? I don't see `id` being used anywhere else.


Sur quelle branche on peut tester ça ? car dans l'idée ça me va


comment ....


Use the JS file from src, too. The dist one has comments stripped.


Assertions/logging in the kernels, no?


On line 15 and 17 I think you should check if you need to add or remove a class or not before you actually do. Otherwise you may be adding classes you don't need and trying to remove classes you don't even need to remove.

Looking this over, I think I forgot to do that on line 29 of my version. Woops.


Nine that I can think of, particularly for tests (where you hope to expect repeatability).

Well, maybe one - you want the user to have a certain experience, like "after I press Enter, the system delays a bit but responds in less than 1.5 seconds". Being time-dependent makes implementing this pretty straightforward, and (mostly) platform-independent (assuming your code can accurately track how long its operations take).

Sent from my iPad

On Jul 22, 2015, at 07:46, Simon Warta <notifications@github.com<mailto:notifications@github.com>> wrote:

Is there any good reason at all to use PBKDF with a time limit rather than a fixed iteration count? I mean, you get different hashing results when you just set a time.

—
Reply to this email directly or view it on GitHub<https://github.com/randombit/botan/commit/30b0e3c88e94ba04c1843798f7ac74a008e01d9b#commitcomment-12296234>.


i'd sort the list alphabetically (as in actually sorting the items here, not `.sort()`)


#73

Pierrick Guyard

On Wed, Jul 22, 2015 at 1:51 PM, Jean-François Garreau - Binomed <
notifications@github.com> wrote:

> Sur quelle branche on peut tester ça ? car dans l'idée ça me va
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GDG-Nantes/devFestSite-2015/commit/75143448784bfde0816ae271d5f6b502f6c727bf#commitcomment-12296326>
> .
>


But fetching too many RunJobs should be up to the server to limit -- we have pagination turned on for runjobs, right?

So something like `fetchRunjobs({params: "workflowrun": wfrun_uuid})` would maybe limit the runjobs to only those that belong to a workflow run, and the server should return a paginated result.


yeah I was, haha I am stupid... ok I will use that function


Hi, great job!, you know how I can disable fade in from the original template? Thanks in advance!


A challenge here is actually to determine which objects are updated by .update method. The method doesn't instantiate the objects, but it only send an UPDATE sql to the backend. It could be an issue that the developer forgets to send an UPDATE message somewhere...

A dedicated websocket.py is definitely required.

On Jul 22, 2015, at 7:50 AM, Andrew Hankinson <notifications@github.com<mailto:notifications@github.com>> wrote:

Those seem like pretty extreme options. Can't you just put the publish_message method in another file and import it from there? Like a dedicated "websocket.py"?

from rodan.websocket import publish_message seems pretty good, doesn't it?

—
Reply to this email directly or view it on GitHub<https://github.com/DDMAL/Rodan/commit/f3a0cbaeca6cbef96e9451db8df5189d64eef01c#commitcomment-12296308>.


@devinus Is there anything one could help, to make this active again?


Ultimately it's not a big deal -- this client is temporary.


Why remove the virtual destructor?


Yeah only this decreases the verbosity of the primary use case


Yes, but still takes time for the client to glean all the pages, and the user impression will be not that responsive when the user checks the box of displaying the run jobs.

On Jul 22, 2015, at 7:52 AM, Andrew Hankinson <notifications@github.com<mailto:notifications@github.com>> wrote:

But fetching too many RunJobs should be up to the server to limit -- we have pagination turned on for runjobs, right?

So something like fetchRunjobs({params: "workflowrun": wfrun_uuid}) would maybe limit the runjobs to only those that belong to a workflow run, and the server should return a paginated result.

—
Reply to this email directly or view it on GitHub<https://github.com/DDMAL/Rodan/commit/ddb1d2276eb1f4be245652db71ca3acdaa7a6ee0#commitcomment-12296356>.


Yes.

On Jul 22, 2015, at 7:53 AM, Andrew Hankinson <notifications@github.com<mailto:notifications@github.com>> wrote:

Ultimately it's not a big deal -- this client is temporary.

—
Reply to this email directly or view it on GitHub<https://github.com/DDMAL/Rodan/commit/ddb1d2276eb1f4be245652db71ca3acdaa7a6ee0#commitcomment-12296372>.


Just FYI, setting this back will never work. You cannot set a passed in parameter to another value and have it reflected in the code that called this method. That is unless you use a 'ref' argument (which is the purpose of ref)


@mcb Have you checked out the Phoenix framework?


> Nine that I can think of

Nine or None? I'm a bit confused, sorry.

>  after I press Enter, the system delays a bit but responds in less than 1.5 seconds

Isn't that called either "cpu-burner" or "sleep"?


Ha, just seeing this now. Thank you :)


I think it should be ```field = *(m + lblattice.nb_offset[i]*lbmodel.n_vel + field_offset);```.
Perhas one can write it like this ```field = (m + lblattice.nb_offset[i]*lbmodel.n_vel)[field_offset];```?


hi,

this code in master is currently broken because the chosen bower repository is still broken.
We have functional code in commit 99e9dd182219013ce43217358a77135f31b00cd0 (release tag 1.0.5-dev2).

The newer versions of chosen and bootstrap are not well tested by us. This is the reason I did not made a pull request for now.


The backslash is unnecessary. And, if used, should be string-escaped.


Rather than repeat these constant definitions here, you can just use the ones from the Battleship class. You just have to fully-qualify hem, so Battleship.MISS, Battleship.HIT, etc.


Remove style attribute although it is empty.


No problem. Glad I could help ;)


@devinus Yes I have, and am not overly happy with it. I was always much happier with the simpler approach that dynamo took, which is why I am asking.


Add ```type="submit"``` please.


I dunno if I like the badges on the heading looks funny


Prepocessor is not exactly the issue, I think. What makes a difference is to have () or not. It will work without parenthesis assuming the adjustment factor given as rational comes at the end. There are very few places where doubles are used in Verovio, and we need them sometimes. But I am not convinced we need this here.

#define CIRCLE_RADIUS_FACTOR 32 / 100

even though I would suspect that what you actually want 1 / 3, and if you want 0.31, you just do 31 / 100 ;)


Could you user a class as ```js-whatever``` please?


je sais pas si on aurait pas intérêt à faire un code couleur plutôt...? car par exemple si softeam nous sponsorise encore en silver, en soit, ils sont historiques mais ils mettent moins de sous sur la table...


Thanks for the explanation. I saw this issue on hpc-coveralls. I'll stay tuned, it's no big deal for a side-project like this to wait a bit.


I noticed ;-)


The key and the secret should not be exposed. It can lead to breaches via X site Scripting. The general method is to take them in from environment variables.


Important moment: we used just positive numbers and 0, not negative.
We can add NegativeControl for bytes, but it will take some time( result x4 faster then standart sort).


`http://loca` - intended or typo?


H5页面窗口自动调整到设备宽度，并禁止用户缩放页面
<meta name="viewport" content="width=device-width,initial-scale=1.0,minimum-scale=1.0,maximum-scale=1.0,user-scalable=no" />
　忽略将页面中的数字识别为电话号码
<meta name="format-detection" content="telephone=no" />
　忽略Android平台中对邮箱地址的识别
<meta name="format-detection" content="email=no" />
　当网站添加到主屏幕快速启动方式，可隐藏地址栏，仅针对ios的safari
<meta name="apple-mobile-web-app-capable" content="yes" />
　将网站添加到主屏幕快速启动方式，仅针对ios的safari顶端状态条的样式
<meta name="apple-mobile-web-app-status-bar-style" content="black" />
　移动端如何定义字体font-family
body{font-family:Helvetica;}中文字体使用系统默认即可，英文用Helvetica
　ios系统中元素被触摸时产生的半透明灰色遮罩怎么去掉
a,button,input,textarea{-webkit-tap-highlight-color: rgba(0,0,0,0;)}
　部分android系统中元素被点击时产生的边框怎么去掉
a,button,input,textarea{-webkit-tap-highlight-color: rgba(0,0,0,0;)-webkit-user-modify:read-write-plaintext-only; }
　winphone系统a、input标签被点击时产生的半透明灰色背景怎么去掉
<meta name="msapplication-tap-highlight" content="no">
　webkit表单元素的默认外观怎么重置
.css{-webkit-appearance:none;}
　webkit表单输入框placeholder的颜色值能改变么
input::-webkit-input-placeholder{color:#AAAAAA;} input:focus::-webkit-input-placeholder{color:#EEEEEE;}
　禁用 select 默认下拉箭头
select::-ms-expand {display: none;}
　禁用 radio 和 checkbox 默认样式
input[type=radio]::-ms-check,input[type=checkbox]::-ms-check{display: none;}
　禁用PC端表单输入框默认清除按钮
input[type=text]::-ms-clear,input[type=tel]::-ms-clear,input[type=number]::-ms-clear{display: none;}
　禁止ios 长按时不触发系统的菜单，禁止ios&android长按时下载图片
.css{-webkit-touch-callout: none}
　禁止ios和android用户选中文字
.css{-webkit-user-select:none}
　打电话
打电话给:0755-10086
　发短信，winphone系统无效
发短信给: 10086
　屏幕旋转的事件和样式 window.orientation，取值：正负90表示横屏模式、0和180表现为竖屏模式； js
window.onorientationchange = function(){ switch(window.orientation){ case -90: case 90: alert("横屏:" + window.orientation); case 0: case 180: alert("竖屏:" + window.orientation); break; } }

　屏幕旋转的事件和样式 window.orientation，取值：正负90表示横屏模式、0和180表现为竖屏模式； css3
//竖屏时使用的样式 @media all and (orientation:portrait) { .css{} } //横屏时使用的样式 @media all and (orientation:landscape) { .css{} }
　audio元素和video元素在ios和andriod中无法自动播放
$('html').one('touchstart',function(){ audio.play() })
　摇一摇功能
HTML5 deviceMotion：封装了运动传感器数据的事件，可以获取手机运动状态下的运动加速度等数据。
　手机拍照和上传图片
 <input type=file accept="image/*">  <input type=file accept="video/*">
　微信浏览器用户调整字体大小后页面矬了，怎么阻止用户调整
body{-webkit-text-size-adjust: 100%!important;}
　消除transition闪屏
.css{
/*设置内嵌的元素在 3D 空间如何呈现：保留 3D*/ -webkit-transform-style: preserve-3d;

/*（设置进行转换的元素的背面在面对用户时是否可见：隐藏）*/ -webkit-backface-visibility: hidden;

}
　开启硬件加速

.css {

-webkit-transform: translate3d(0, 0, 0);

-moz-transform: translate3d(0, 0, 0);

-ms-transform: translate3d(0, 0, 0);

transform: translate3d(0, 0, 0);

}
　取消input在ios下，输入的时候英文首字母的默认大写

<input autocapitalize="off" autocorrect="off" />
　android 上去掉语音输入按钮

input::-webkit-input-speech-button {display: none}
　如何阻止windows Phone的默认触摸事件

html{-ms-touch-action: none;}/* 禁止winphone默认触摸事件 */
　要做到全兼容的办法，可通过绑定ontouchstart和ontouchend来控制按钮的类名

<!DOCTYPE html>

<meta charset="utf-8">
<meta content="width=device-width,initial-scale=1.0,maximum-scale=1.0,user-scalable=no" name="viewport">
<meta content="yes" name="apple-mobile-web-app-capable">
<meta content="black" name="apple-mobile-web-app-status-bar-style">
<meta content="telephone=no" name="format-detection">
<meta content="email=no" name="format-detection">

按钮

　模拟按钮hover效果

移动端触摸按钮的效果，可明示用户有些事情正要发生，是一个比较好体验，但是移动设备中并没有鼠标指针，使用css的hover并不能满足我们的需求，还好国外有个激活css的active效果，代码如下，兼容性ios5+、部分android 4+、winphone 8
<!DOCTYPE html>

 <meta charset="utf-8">
 <meta content="width=device-width,initial-scale=1.0,maximum-scale=1.0,user-scalable=no" name="viewport">
<meta content="yes" name="apple-mobile-web-app-capable">
 <meta content="black" name="apple-mobile-web-app-status-bar-style">
<meta content="telephone=no" name="format-detection">
 <meta content="email=no" name="format-detection">

按钮

移动端字体单位font-size选择px还是rem
html {font-size:10px}
  @media screen and (min-width:480px) and (max-width:639px) {
  html {
  font-size: 15px
  }
  }
  @media screen and (min-width:640px) and (max-width:719px) {
  html {
  font-size: 20px
  }
  }
  @media screen and (min-width:720px) and (max-width:749px) {
  html {
  font-size: 22.5px
  }
  }
  @media screen and (min-width:750px) and (max-width:799px) {
  html {
  font-size: 23.5px
  }
  }
  @media screen and (min-width:800px) and (max-width:959px) {
  html {
  font-size: 25px
  }
  }
  @media screen and (min-width:960px) and (max-width:1079px) {
  html {
  font-size: 30px
  }
  }
  @media screen and (min-width:1080px) {
  html {
  font-size: 32px
  }
  }


Yes you are right. Thanks for the insight. I have reverted my commit!


@mcb That's unfortunate, but I really don't think there's any reason to continue working on Dynamo with Phoenix available. Phoenix is not only maintained and well documented, but it uses Plug and core Elixir contributors all work on it.


Since `__del__` and `close` are effectively identical, let's have one call the other.


We can extract it from config to Attachment class. Just keep ONE place for all this backend-related path building.


Remove this code (113-118) too


should be space after 'new'


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

VSO State: Done


I think you should have a second Battleship constructor which auto-defaults to an empty grid if one isn't supplied, so that you can do Battleship.new(ROWS, COLUMNS)


Why this duplicate line?


mispelled canola


Nevermind, I read to fast. Sorry


Readme should be agnostic of the branch and authors.


why?


data-username better?


that's why there is a fork option in git :)


How were you able to compile this and I wasn't? Was the previous syntax valid using your compiler and invalid in mine (VC++)


@thecodeassassin Yeah...that's right. Maybe we could just add a PointsBuilder to create them in a more readable way


What is the Grid compared to a Battleship?
I think the names need to be sorted out. What behaviour does Grid have? Maybe that would give you a better idea.


What is this group?


Exit break all the code. If I have other errors it will not show.


If we do it right, postgresql triggers might be the right way to go.

http://gonzalo123.com/2012/11/26/sending-sockets-from-postgresql-triggers-with-python/

We should definitely dig into the Redis bit -- it may be that we can get PostgreSQL trigger Redis directly without needing to process it through Rodan.

In an ideal world we can attach the triggers to the model tables when they're created (`post_syncdb`) and then they should "just work," sending any messages on create, update, or delete over websockets with minimal processing in Rodan itself.


need something less commonly used.
ex: ```_GIT_VERSION_```


backend도 backent로 되어있네요 ㅠㅜ


I would bet some money they don't link the media server with the chat server. Those guys are really dumb.
I've tried lots of user agents in the past and pretty much everything works.


I think this is a typo: "we cannot play limits on the" should read "we cannot put limits on the" or "we cannot place limits on the"?


I'm ok with this idea. Note that sockets are automatically closed upon garbage collection, so while it's good to efficiently manage resources, it's not a critical problem in the big scheme of things.

However, while this hasn't been as much of an issue in the past, it should be noted that this is not thread-safe.


beware of merge conflicts


I'd prefer to see the bulk of this comment in the commit message for this patch. The comment itself can just be:
`# Shutdown and close the socket properly.`


@alloy  Thank you for your comment. I never knew that :p


ack


Perfect !!! Thank you for your correction :-)


1: M111 unrecognized component name: utils


Looks good to me.


Well `--` is forbidden in XML comments, so yes they're required.


what's that newline doing here?


That's how it always was, check the eclipse call below.


ah, I see. thanks.


Please update the commit message with the comment in the `close` method. While you're at it, go ahead and format the commit message as follows:
`Single line description, usually no more than 50 characters

Paragraph description going into further detail.`

This format isn't officially defined in any PyKMIP documentation but its something I try to follow. I'd like to standardize it eventually.


Oh, sure, it was a previous placement hack :sweat_smile:


huh. okay… thanks


Ok.


Feel free to submit pull request fixing it. Thanks!


FYI writing to the filesystem is borked with the latest kernel (I saw you make an `appointment-storage` branch).


I'm a bit stupid right now, why do we want that the `config` setting *is set*? Don't we want to do the find config stuff if it is **not** set?


Don't you like `cdb` prefix? I don't know if we're using `data-*` for our purposes, but I like namespacing (all the) things. `current_username` is certainly better, changing it.


o kurde, miało być v4 xD


Yup because the light style isn't default


why the indentation change?


I'd prefer to have those changes in multiples commits, It would be easier (for me, at last :) to review and merge.

For the unused entity problem, I'd prefer its usage to be fixed, instead of dropping the option

For the multiple host feature, I'd prefer to accept multiple occurence of --host option, instead of parsing its value. Otherwise, that's a great idea.


God i hate that semicolon fest here.


The email used to (user.email) commit wasn't attached to your account at the time of posting. You didn't have the email set, so it used your macbook's username + hostname (`msherman@nymshermanmbp13.local`) instead of email address.


Relic from earlier - forgot about it


Ah yes.


Sure. This came from `./public/assets/3.17.11/javascripts/organization.js:    $('.OrganizationUser-renewApiKeyLink').bind('click', function(ev) {`.


It's done


But where should I put the `getParents()` method?


You also can't rectify that one without altering history and ruining everyone's forks/cloned copies.


No problem, let's add a 'js-thingy' class :).


This is not a change in how we hash things, just that for p2 generation I needed an md5 hash method and this was the easiest place to put it in.


Then it may should be a warning. It's a problem but it should not crash the wiki /  the rendering of the document.


You've included extra files (*/data/config) for some reason.

(Also, please set your editor to produce EOF newlines.)


IMO namespace variables is great when it is a common tool in the environment, but as data-* thing is not a recurrent way. I don't know, happy with any decision you take :)


extra whitespace


This should be `cabal sandbox init`.


@hvelarde this check is to know if the URL is an attachment or internal post.  Without it, it will create a resolveuid url for posts too, right?


Please remove


@RandomAltThing What's wrong with your github account


Imports are out of order.


We'll look into it. Thank you!

On Jul 22, 2015, at 8:13 AM, Andrew Hankinson <notifications@github.com<mailto:notifications@github.com>> wrote:

If we do it right, postgresql triggers might be the right way to go.

http://gonzalo123.com/2012/11/26/sending-sockets-from-postgresql-triggers-with-python/

We should definitely dig into the Redis bit -- it may be that we can get PostgreSQL trigger Redis directly without needing to process it through Rodan.

In an ideal world we can attach the triggers to the model tables when they're created (post_syncdb) and then they should "just work," sending any messages on create, update, or delete over websockets with minimal processing in Rodan itself.

—
Reply to this email directly or view it on GitHub<https://github.com/DDMAL/Rodan/commit/f3a0cbaeca6cbef96e9451db8df5189d64eef01c#commitcomment-12296681>.


Not used.


hello. can you make to upload and download progress status?


I think we need to solve the UID first and then we deal with the paths


throw IllegalArgumentException if metadataLoader is null


I was just thinking about that, I think that the Grid should just display the grid, so I'll move the parts that don't handle display/input/output into the Battleship class ...


I think this class will need to be refactored to include tasks that want to depend their execution success on some value they have arrived on after execution later on.
For now and for the sake of finishing for the initial version release, this is okay enough.


Formatting.

It may be a good idea to extract a utility method for progress reporting.

Avoid using ```String.format``` in such simple cases - ```"literal " + value``` is much easier to read.


As predicted in [my comment](https://github.com/time1015/elevator-simulator/commit/b7871fa0ed8415b46baf3fa2e3802753f812221f#commitcomment-12296911), this class may have to be refactored as well.


Use ```NAME``` constant.


Formatting.


The original code had a test here to exit early if !countryCallingCodeToRegionCodeMap.containsKey(countryCallingCode).

Can you look into whether removing it is necessary and what the consequences are?


Long line.


Do not start methods with blank lines. What's the point?


Well I think three lengths (eg 6, 25, 40) would suffice for testing purposes :wink:


Nice, it'd be good to add negative test too


(Please remove unused imports in this file too.)


"get" or "compute" would be more accurate. "receive" is as if you were passively waiting for dirty files to arrive.


I think it's better if this is just "new MultiFileMetadataSourceImpl(DEFAULT_METADATA_LOADER)" and the default META_DATA_FILE_PREFIX is set within that class.


@vangheem This throws

    patterns.pat-structure: Error initializing button Delete SyntaxError: expected expression, got '<'


Maybe it can be reason of apps force closes so you can try new build and see if it fixes force close issues


What about the Reaction.ml file ?


Maybe we can move the registerNib code here, so the cell design can be swapped


Ok, then please forget about my remark.


Remove "If null, default metadata loader will be used." - this isn't true.


전체적으로 파일의 맨마지막에 줄바꿈이 빠져있네요..


I believe this line is over length.


Remove accepting `arguments` was the original comment


And I continue seeing no reason for state. I also have to remember that `204` is *No Content*, by having state you're introducing bugs such as an attempt to `Client#getResultSet` when a write operation was performed and it's has no results. You never ever seen a *resultSet* for a **INSERT** statement.