> I thought that running `rake generate:html` and `rake generate:latex` generated the reference examples.

It doesn’t make any sense to test what you’ve just generated. Generator is just a helper to get output examples quick. Then you must review them (!) and commit to the repository, so when you break something in the future, you will know it because the test will fail. Tests _obviously_ can’t fail (unless converter throws some error) when they’re run against freshly generated examples, the output will be always the same.


Do note that this is only Atum ores. Not anywhere else.


Hate to do this but have you considered `DeferredCredits`

/ducks


@nbp can you elaborate on the reason for this? I think that reducing my elusive infinite recursion is related to this, since I see a similar pattern in https://github.com/NixOS/nixpkgs/blob/15bb4c20e614ed6835c59c7c9101ee59eacbe473/nixos/maintainers/scripts/ec2/amazon-hvm-config.nix and I use that in my generated AMIs.


I don't think that will work, at least not in the way I imagined.

In fruitmachine we want to have fastdom instance per component, so when component is removed, we can cancel all pending fastdom writes.

However, we still want all components to share the same rAF.

So there probably needs to be a global queue of all scheduled fastdom instances, and flush must go through all of their read queues before going through all write queues.


Go, Emma, go!


`BEAST_DEFINE_TESTSUITE (CachedCredits, ledger, ripple);`


please rename your file to faize.cpp or some name that is you all about you.  add a .h file with the same name containing a prototype for the function above.  my idea is that any other contribution you make can go here also and your efforts easily seen.  If your to shy, then I / we can do something a little less overt.


I do understand that. I have been using doctest to incrementally test the changes as I make them.  That has
been very helpful.  But how do I get back to a state where I can see all the broken things that you see?


What would `deferResult(success: false)` mean?  I think you just want `Deferred<Result<()>>`, which @rnewman abbreviates to `Success`:
```
Sync/Synchronizer.swift:13:18:public typealias Success = Deferred<Result<()>>
```


Throughout, use `deferResult(success: value)` to streamline this.


it is coming from String class.  This line's technical syntax is `result.*("\n")` but Ruby gives us some sugar for the method so it looks like a mathematical operation.


Is there a reason why we return the very first project here? As getAccessibleProjects returns multiple entries.


Really nice work, not what I was thinking. Damn I love collaboration.  You can learn so much from others.


This isn't hitting the UX mocks at https://mozilla.invisionapp.com/share/J427EH8VB#/screens/64313418?maintainScrollPosition=false but that could be a follow-up.


Are you referring to `fastdom.sandbox()`? `Sandbox` uses the read/write API of the `fastdom` instance passed to it so we never end up with more than one `rAF` loop.


dmd2.065 does not compile this:
```
SDLang-D/src/sdlang/token.d(271): Error: template std.array.split cannot deduce function from argument types !()(Duration)
....
```


```
flush() {
  try {
    runTasks()
  }
  finally() {
    scheduleFlush()
  }
}
```

and then:

```
if (this.catch) try { flush() } catch(e) { this.catch(e) } else flush();
```

This way you don't need to rethrow and Chrome won't destroy the stack trace.


This is incorrect. You need to do this:

```
var result: SQLiteDBConnection?
dispatch_sync(…) {
  …
  result = self.sharedConnection
}
return result!
```


You might just want `Deferred<Bool>`, since there's no way for a clear operation to not complete.


What do you see [here](https://github.com/asciidoctor/asciidoctor-latex/tree/master/test/examples/tex)?


Actually, just

```
deferResult(value)
```

— if `value` isn't an `ErrorType`, this will take the success branch.


Thanks, I'll make that change.


Prefer `self.queue = …`.


@tianon Following up on this issue. I would also like to understand the rationale of this change. Was there a specific scenario that this was intended to address? Given that we still depend on MYSQL_PORT_3306_TCP (to check linking), why not just use it for discovery?

In our scenario we need the capability to link arbitrary containers. As a result we want to only depend on standard environment variables. This change would break our usage.

Thanks a bunch!


@jtaylor-sfdc, this was the bug in my PHOENIX-900 impl that caused a bunch of test failures related to deletes. Want to run this by you quickly before committing.


Is there already a private identity function I can use instead?


Is this file the best place to put these functions?


You're right! Make sense!


You can see [this test](https://github.com/wilsonpage/fastdom/blob/40fc440c6bfb8929613228447d0bb2c4b8357dfa/test/test.js#L346) as an example of how Sandbox works.


Integration test has been added to test this behavior


These are the colors of the lines on the screen. I was thinking about doing a set of globals so the user can change it.


This will return the journal mode; we should ensure that it's `"wal"`, and scream if it's not.


no comma


in globasl prefs


These changes look much better, but by pushing the base patch (after a rebase?) I can't see my old comments.  Certainly some of the old issues appear to remain (for example, there was a nit about assigning and not using an `index` variable). :(

This doesn't need to be perfect, I guess -- we can improve things over time.


I *assume* that ember tests are run in Travis against things like IE9. However, I'm unsure where to add a unit test that tries this against an SVG element. This may also fail in IE8 or other browsers that simply don't support SVG... Unsure how or where to proceed.


Remove the comments that were left from testing/coding


libcxx_ubuntu_14_04 (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (337 of 337 tests passed)

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


why are you turning the tuple into a set, then turning that into a dict?


please reschedule your test based on the new master, and pay close attention to the diff. you are mixing your patch with the rook contact check stuff that was already merged.


rax retry tests


merge two activities


why paperclip instead of carrierwave?


@sprakes I just wanted to inform you that we have a fast heap-based priority queue data structure which is actively maintained by @kittenRainbow  and @CalvinBench. This is being used by the scheduler. You can find it here: https://github.com/rellermeyer/course_os/blob/process_control_scheduler/kernel/data_structures/priority_queue.c

It is bundled with test cases and all. Functionality is being added to it as well.


@pholleran Are there ever cases where lat and long could <= -90 or -180?


在Ubuntu下，无论文件编码怎么改都是乱码，怎么解决？
文件的编码已经是GBK了，
-master $ enca -L zh_CN abc.txt
Simplified Chinese National Standard; GB2312


Looking good so far.  Does the example actually work?  Can you post some of the output?  I don't have a build of Tao currently, was it fairly easy to configure/build with PETSc?


I propose this line instead:
```
if (_screenTypeObject && _screenTypeObject.name == type) {
```
This line will check for the existence of the `_screenTypeObject`, _and_ it's the same as the one we're requesting, we return it. If not, we fetch the object using the search below. This way we don't need the cache value. And we can drop all of the arguments on line 241. Drop `ignoreCache` entirely.

This should achieve the same caching effect, but be smarter about it than the original implementation.


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


yes


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (657 of 658 tests passed)

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


Oh hell. Sorry, I'm really bad at Github. :P


Yep, the example works. If I use the options in run.sh, then I get the output below:

    iter =   0, Function value: 0,  Residual: 0.217778
    iter =   1, Function value: -0.281146,  Residual: 1.41941e-06
    Tao Object: 1 MPI processes
      type: nls
          Rejected matrix updates: 0
          Newton steps: 1
          BFGS steps: 0
          Scaled gradient steps: 0
          Gradient steps: 0
          nls ksp atol: 0
          nls ksp rtol: 1
          nls ksp ctol: 0
          nls ksp negc: 0
          nls ksp dtol: 0
          nls ksp iter: 0
          nls ksp othr: 0
      TaoLineSearch Object:   1 MPI processes
        type: more-thuente
      KSP Object:   1 MPI processes
        type: stcg
      total KSP iterations: 28
      convergence tolerances: fatol=1e-10,   frtol=1e-10
      convergence tolerances: gatol=1e-08,   steptol=0,   gttol=0
      Residual in Function/Gradient:=1.41941e-06
      Objective value=-0.281146
      total number of iterations=1,                          (max: 50)
      total number of function evaluations=7,                  max: 10000
      total number of gradient evaluations=2,                  max: 10000
      total number of Hessian evaluations=1
      Solution converged:   estimated f(x)-f(X*) <= fatol
    Converged reason: 1
    Tao converged.

Regarding a Tao build: It's bundled with PETSc 3.5, so if you use that then you don't have to do any extra installation (very convenient!).


can you use more descriptive variables?


Related to #2 implementation,

Next step is the preference screen implementation


Why is this necessary? Could you point out what error this is causing?

Also, since a '\0' terminated string is needed, you'll need to call name.c_str() or name.data().


@bluejamesbond Hey, thanks. That's why were messing with stuff on a different branch! I deleted priority_queue.h in the pcb_exec branch on account of not knowing where it was being used. When you guys go stable could you merge an update with master? It'll be useful to be able to pull the latest pqueue stuff from you guys.

(Also for @kittenRainbow @CalvinBench) I also changed the function names in priorityQueue.c/.h to pqueue_add(), pqueue_init(), etc since otherwise it would be difficult to use them in conjunction with any other library. That change is currently on the pcb_exec branch, let me know if that's okay with you guys. If so either I can push that one change by itself to your branch or one of you can manually do it.


Hi, I would like to know when this fix will be ready to download as 0.8.1 version?

Thanks


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

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


Hm. Do you know why this happened with my pattackcount2 test but not my connected_simplify one? Both are tested against the same base, and that master has the rook contact check stuff merged.


This appears to resolve this ticket: http://trac.cython.org/ticket/687


@sprakes We will rename all the functions to prefix with `pqueue` instead of `prq`. However, I don't think we will be able to merge until the scheduler is completed. The priority queue in our branch is a generic one. The scheduler.c file in our branch builds on the logic from last year and uses the generic priority queue that we have.

The files I am referring to are here:
- [priority_queue.c](https://github.com/rellermeyer/course_os/blob/process_control_scheduler/kernel/data_structures/priority_queue.c)
- [scheduler.c](https://github.com/rellermeyer/course_os/blob/process_control_scheduler/kernel/process/scheduler.c)

Let us know if you have questions or concerns.


You can install specifying this commit

    "ember-simple-auth": "git@github.com:simplabs/ember-simple-auth.git#523e677a80522d383a445081488c0f3d7e5b938d"


Why do all the <input> elements have a backslash at the end of them?


I have commited


test comment


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


I've been implementing things in klibc.h since everything uses that header.

I don't know if you saw the commit the other day I pushed to master but everyone can use assert() now. Functionality is assert(<<boolean expression>>). It'll kernel panic and spit out the file, line number, and whatever the expression was if the assert fails.

You can also && a string with your expression for verbosity (since it just creates a tautology). It's basically a janky way to have verbose assets. Ex: assert (1>2 && "is false");


:( I'll be here if you return.


:+1: - we've already reviewed this internally.


Can be removed, does nothing


required以外も、このように共通化できるかなと思いました。(動かなかったらごめんなさい)
```php
     private $messages = array(
            'required'          => ':attributeを入力して下さい',
            'max'               => ':attributeを:max文字以内で入力して下さい',
            'zenkaku'           => ':attributeを全角で入力して下さい',
            'gender.required'   => ':attributeを選択して下さい',
            'digits'            => ':attributeを正しく入力して下さい',
            'pref_id.prefcheck' => ':attributeを選択してください',
            'email'             => ':attributeを正しく入力して下さい。',
            'required_with'     => ':attributeを入力して下さい．'
    );

    private $names = array(
            'lastname'       => '姓',
            'firstname'      => '名',
            'gender'         => '性別',
            'postcodeFirst'  => '郵便番号',
            'postcodeSecond' => '郵便番号',
            'pref_id'        => '都道府県',
            'mailaddress'    => 'メールアドレス',
            'hobbyOtherText' => 'その他の詳細'
    );
```


I would use framework.Equals(FrameworkConstants.CommonFrameworks.DnxCore50)


Hmm. Then what is [priorityQueue.c](https://github.com/rellermeyer/course_os/blob/811e36d1297aa0fc7fc9cfd9e5bc838ab6a340a7/kernel/priorityQueue.c)? I'm confused as to what this one does if you guys are going to be using priority_queue.c instead.


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1656 of 1676 tests passed)

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


Still need to figure out anchors and better positioning to keep the meter attached to the tank sprite.


So right here do you mind providing a little more info on what's going? So from my educated guess we are modifying the first level pt addresses and enabling the coarse_page_table address which in turn enables 2nd level paging?


This throws a NPE if settings.get("transport.type") returns null. According to the comment, this setting should be optional, here is the error I am getting:

java.lang.reflect.InvocationTargetException
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.xbib.elasticsearch.plugin.jdbc.feeder.Runner.main(Runner.java:45)
Caused by: java.lang.NullPointerException: null value in entry: transport.type=null
        at org.elasticsearch.common.collect.CollectPreconditions.checkEntryNotNull(CollectPreconditions.java:33)
        at org.elasticsearch.common.collect.RegularImmutableMap.<init>(RegularImmutableMap.java:88)
        at org.elasticsearch.common.collect.ImmutableMap.copyOf(ImmutableMap.java:294)
        at org.elasticsearch.common.settings.ImmutableSettings.<init>(ImmutableSettings.java:70)
        at org.elasticsearch.common.settings.ImmutableSettings$Builder.build(ImmutableSettings.java:1134)
        at org.xbib.elasticsearch.plugin.jdbc.feeder.JDBCFeeder$2.create(JDBCFeeder.java:291)
        at org.xbib.elasticsearch.plugin.jdbc.feeder.JDBCFeeder.createRiverFlow(JDBCFeeder.java:156)
        at org.xbib.elasticsearch.plugin.jdbc.feeder.JDBCFeeder.readFrom(JDBCFeeder.java:142)
        at org.xbib.elasticsearch.plugin.jdbc.feeder.JDBCFeeder.exec(JDBCFeeder.java:119)
        ... 5 more
Exception in thread "Thread-0" java.lang.NullPointerException
        at java.lang.Throwable.printStackTrace(Throwable.java:653)
        at java.lang.Throwable.printStackTrace(Throwable.java:643)
        at org.xbib.elasticsearch.plugin.jdbc.feeder.JDBCFeeder$1.run(JDBCFeeder.java:227)


is this remaining?


I believe the framework just needs to be compatible with portable-net45+win8, which isn't a real profile. You might want to check the old dnx code that was handling this before, and then I can show you how that could work with NuGet.Frameworks.


+1 for providing an example.


I wrote them by myself


File Name modification


oh... thanks!


Fuck it, I'll just create a new branch and use that.


i like this


```dal.js``` isn't being used in this file from what I can tell. A require is only needed in `sla.js`.


same error.


Why not merge your modifications in https://github.com/rpcs3/rpcs3


> Yep, the example works. If I use the options in run.sh, then I get the output below:

It converges in 1 optimization iteration?  I guess this is expected for the particular problem (SPD matrix A, etc.) that you set up...

> Regarding a Tao build: It's bundled with PETSc 3.5, so if you use that then you don't have to do any extra installation (very convenient!).

Agreed.  Still seems like we need a reasonable configure test for whether Tao is enabled by PETSc and a separate preprocessor token, e.g. LIBMESH_HAVE_TAO before this can be merged; I can help with either or both of those. I'm in favor of merging it as soon as you are ready...


Hi

The line $tmp = dirname($current_css_file).'/'.$matches[2]; in classes/Tools.php is redundant because $tmp is defined 2 lines later and this definition is not referenced.

Other than that the code looks good although I have not yet had opportunity to test.

Regards


i like rantai sinnob untuk vixion


I'm still not sure what you mean?


It should be a straight `--add`, because `$hash` isn't used and we directly get back the path.


In PR 19, there was the task to combine ```getAllTrunks``` into the ```trunks``` attribute and ```getAllStations``` into the ```stations```.

I understand this is a subset of the total intended changeset, so, I only note that here so that it does not fall off the map.


Well, the button clearly says "EDIT pedigree". So you kind of explicitly approve to leave view mode and switch to edit mode. Just by playing with pedigrees myself and checking what other people did, it is very annoying to first press edit, then expand family section, and only then press "edit pedigree". Given pedigree editor now saves data back to patient records it is basically just another editor interface, less capable for some things but more convenient when working with multiple patients (e.g. see all at once or drag disorders between patients). So adding a shortcut link to edit pedigree to the view patient page IMHO makes perfect sense. Closing pedigree would go back to the view mode if invoked from the view, and to edit mode if invoked from the edit mode. ALl those modes are separate but IMHO any transition i\makes sense (e.g. "view patient -> edit pedigree -> view again" or "edit patient-> edit pedigree -> back to edit"


This is backwards. You want to clear the errors before trying to read instead of after. You'll also want to clear the errors before trying to seek.


@barbeau This might be minor, but I notice that ```DO NOT MANUALLY EDIT THIS FILE``` message is not very noticeable in the IDE:
![screen shot 2015-04-20 at 11 40 04 pm](https://cloud.githubusercontent.com/assets/2777974/7245331/46477c5c-e7b7-11e4-87fd-bc637a1b8323.png)

Maybe putting one extra star in the comment message would it make more noticeable:
```
/**
+ * DO NOT MANUALLY EDIT THIS FILE
+ *
...
```
 ![screen shot 2015-04-20 at 11 46 46 pm 2](https://cloud.githubusercontent.com/assets/2777974/7245349/a17f679c-e7b7-11e4-9d73-754faca0faab.png)


I used "NLS" (i.e. Newton with Line-search). Since we have a quadratic objective function, Newton will converge in one iteration (Newton solves for gradient = 0, and the gradient is linear). So it's a pretty trivial example, but seems like a good choice for optimization_ex1 to me.

I'm ready to merge this. There's obviously tons more to do on this optimization functionality, but it'd be good to do that via PRs into the main libMesh repo.

So if you can help with LIBMESH_HAVE_TAO etc so that we can get this in good shape to merge, that'd be great.


I forgot that input fields don't need the closing tags when I first created these forms. Would you mind removing them from -`users.create.php` and -`users.edit.php`, then re-commit/push before we merge with the master branch?


Alas, Ruby doesn't have a built-in notion of an "enum."


这句英文的可以删掉。


Beginning to understand better now and so resubmitting this for the last hw assigment.


这里 转向  翻译成 页面跳转 会不会更通顺一些？


Как тебе нравится got, неужели asker настолько плох?


@peterbell, why my link/image functions are not working?


> So if you can help with LIBMESH_HAVE_TAO etc so that we can get this in good shape to merge, that'd be great.

OK, I'll try and have a look tomorrow, will make a PR to your branch if I come up with anything useful.  BTW, did you build PETSc in any particular way (--enable-tao or something)?


Hmm, I don't like that you have multiple methods implicitly depending on the `ROMAN_TO_ARABIC_MAPPING` constant.  Consider passing in the mapping as an argument to some of those lower-level methods.


beat me to it


There's no reason to use `-=` here.  You could just return

```ruby
number - ROMAN_TO_ARABIC_MAPPING[roman_symbol] * num_roman
```


new branch is good. test validated.


我这边 chrome 是有这个功能的，mac 41.0.2272.118 (64-bit)，看了下 canary 也是有的。可以去掉译注


Use `unless EXPR` instead of `if !EXPR` for a single-branch conditional.


@peterbell, sorry for the duplicate message - in this version I had links and images, but neither is showing. :-1:


OK, sounds good.

I didn't build PETSc in any particular way, and I don't think there's any --enable-tao option (or at least I didn't see one in PETSc's ./configure --help). I believe TAO is just part of the default PETSc configuration now.


now it seems make sense :)


I would prefer if we do not abbreviate sandbox. Why not just use sandbox as is?


On Mon, Apr 20, 2015 at 03:55:06PM -0700, daurnimator wrote:
> ah ha. we'll have to wrap it in another function and lua_pcall it then.
>

Doesn't Lua panic if it runs out of memory? It might not be able allocate
space for the string message.


This seems somewhat unrelated, and I think it breaks a few WebAppSec specs. See the failures starting ~6 hours ago at https://travis-ci.org/w3c/webappsec/pull_requests

Reverting this patch fixes it locally, but I haven't dug in to see if this line is relevant.


(Reverting this line fixes the problem. I'll send a PR.)


Builds ok after you did that, but now no longer starts citing

themusicgod1@Hedy:~$ XPCOMGlueLoad error for file /home/themusicgod1/palemoon/Pale-Moon/obj-x86_64-unknown-linux-gnu/dist/bin/libnssutil3.so:
/home/themusicgod1/palemoon/Pale-Moon/obj-x86_64-unknown-linux-gnu/dist/bin/libnssutil3.so: cannot open shared object file: No such file or directory
Couldn't load XPCOM.

error


What's this for?


Missing newline at end of file.


Did you mean .Close() based on the name?


I like my if statements to be indented, even if it means the resulting html is indented wrong.

    {% if title %}
        stuff
    {% endif % }


scope.none -> []


I love Python.


Small caps wasn't working in Chrome with `font-variant-caps`. I use small caps with the abbreviation MFD.


测试


You cannot hold any locks while invoking walker function because the walker function may in turn(although not common) invoke back libnetwork api and cause a deadlock. Make a copy of network map holding the lock and walk that without holding the lock.


@ziogaschr It's fixed in master branch. I had reviewed the pull request and fixed it before i merged the pull request.
Take a look here https://github.com/xmartlabs/XLForm/blob/fb653663e84a5f7ebd0512a2ede2a9ef44a66c48/XLForm/XL/Cell/XLFormSwitchCell.m#L38


Same as above in terms of holding the lock.


I agree and I really like your explanation. I'll keep this in mind when I work on the next part.


saw approval from macintux
at https://github.com/basho/riak_kv/commit/d52c32dfbdd605df05e70d36145932e39469be99


merging basho/riak_kv/fix/az/assorted-dialyzer-issues = d52c32df into borshop-integration-1104-fix/az/assorted-dialyzer-issues


basho/riak_kv/fix/az/assorted-dialyzer-issues = d52c32df merged ok, testing candidate = 09a81014


Pardon the typos in my commit description. (Yet again)


部署


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1656 of 1676 tests passed)

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


awesome. totally understandable now.


Also changed spreadsheet sheet extensions to ".axissheet" rather than ".axis", to avoid potential conflicts with user list and sheet list files.


@sprakes Ill do that as well. Thanks!


I copied this from another similar path. Should I remove it everywhere?


It looks to me that this isn't legal (tested from `step9` branch using `start-noderepl.sh`; Node.js v0.11.13; clojurescript 0.0-2843):

```clojure
ClojureScript:cljs.user> (def people [{"name" :foo} {"name" :bar} {:name "baz"} {:not "name"}])
[{"name" :foo} {"name" :bar} {:name "baz"} {:not "name"}]

ClojureScript:cljs.user> (def people-js [#js {"name" :foo} #js {"name" :bar} #js {:name "baz"} #js {:not "name"}])
[#js {:name :foo} #js {:name :bar} #js {:name "baz"} #js {:not "name"}]

ClojureScript:cljs.user> (map "name" people)
/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:16835
$IFn$_invoke$arity$1 ? f.cljs$core$IFn$_invoke$arity$1(G__9003) : f.call(null,
                                                                    ^
TypeError: undefined is not a function
    at /Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:16835:88
    at cljs.core.map.map__2 (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:16836:3)
    at /Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:10860:129
    at cljs.core.LazySeq.sval (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:10861:3)
    at cljs.core.LazySeq.cljs$core$ISeqable$_seq$arity$1 (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:10952:10)
    at Object.seq (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:4193:13)
    at Object.pr_sequential_writer (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:28557:14)
    at cljs.core.LazySeq.cljs$core$IPrintWithWriter$_pr_writer$arity$3 (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:29236:18)
    at Object.pr_writer_impl (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:28762:12)
    at Object.pr_writer (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:28861:18)
nil

ClojureScript:cljs.user> (map "name" people-js)
/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:16835
$IFn$_invoke$arity$1 ? f.cljs$core$IFn$_invoke$arity$1(G__9003) : f.call(null,
                                                                    ^
TypeError: undefined is not a function
    at /Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:16835:88
    at cljs.core.map.map__2 (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:16836:3)
    at /Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:10860:129
    at cljs.core.LazySeq.sval (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:10861:3)
    at cljs.core.LazySeq.cljs$core$ISeqable$_seq$arity$1 (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:10952:10)
    at Object.seq (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:4193:13)
    at Object.pr_sequential_writer (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:28557:14)
    at cljs.core.LazySeq.cljs$core$IPrintWithWriter$_pr_writer$arity$3 (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:29236:18)
    at Object.pr_writer_impl (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:28762:12)
    at Object.pr_writer (/Users/sumbach/src/cljs/cljs-workshop/out/cljs/core.js:28861:18)
nil

ClojureScript:cljs.user> (map :name people)
(nil nil "baz" nil)

ClojureScript:cljs.user> (map :name people-js)
(nil nil nil nil)

ClojureScript:cljs.user> (map #(.-name %) people)
(nil nil nil nil)

ClojureScript:cljs.user> (map #(.-name %) people-js)
(:foo :bar "baz" nil)
```


@gautham4 it's impossible to review this - the commit consists of the whole of tika. Can you please add a specific commit that only adds your Parser? Some specific comments:

1. Add ALv2 license header
2. Add in javadoc (remove default documentation on top of the class)


@sprakes Yes absolutely!


oh, something to note: you don't have to immediately initialize a constant with a value e.g.
```java
final int HEY;
if (thing) HEY = 100;
else HEY = 25;
```


why not just declare this in the member variable? shouldn't need the constructor?


Heh. This is why I check `$COLORTERM` if `$TERM == xterm` in my shell's rc files: https://github.com/auscompgeek/ohmyzsh-custom/blob/master/10-set-term.zsh

Saves me the trouble with everything that might use 256 colours, including vim :smile:

Alternatively, setting `&term` in vim is less... hacky.


The scale should be given to the draw call as before, not integrated here. If we are adding it here, it completely defeats the purpose of the scale factor in the first place -- we might as well just ask for a width and height.

Now, if the scale wasn't scaling the sprites, that's a different problem (that we can research); but we shouldn't implement it ourselves and void the behavior entirely.


This causes #55, any reason to change this?


if the `for` loop only runs once (from 0 to <1, which would be 0),
does it really need to be there?


You didn't even run it, much less test it?


Thanks for this update. Will merge this to the main stream. Also, can you please tell me your creds? :)


Looks good. Thanks for your change!


main case of issue #7


probably not that important, but in the other two `action` conditions it prints "SKYRIM attacks!" and here it doesn't.
aka continuity error


Here's an old implementation of mine I dug up:

```ruby
# Try this on the CLI: http://cl.ly/image/0p1v2N3m1z17

def baseify(base_map, num)
  # Make sure we're processing the map from largest key to smallest key
  base_map.sort.reverse.reduce("") do |result, (place_value, numeral)|
    count = num / place_value
    num   = num % place_value

    result << numeral * count
  end
end

def to_roman(num)
  roman_map = {
    1000 => 'M',
    900  => 'CM',
    500  => 'D',
    400  => 'CD',
    100  => 'C',
    90   => 'XC',
    50   => 'L',
    40   => 'XL',
    10   => 'X',
    9    => 'IX',
    5    => 'V',
    4    => 'IV',
    1    => 'I'
  }

  baseify(roman_map, num)
end

if __FILE__ == $PROGRAM_NAME
  puts to_roman(ARGV[0].to_i)
end
```


**Motivation**: be able to run project's configuration commands before P4 command is executed.
**Example**:
 * Automatically deduce current workspace (e.g. based on buffer's directory) and set _*${P4CLIENT}*_ environment variable up;
 * Keep a history of run P4 commands.


There's no "newline" at the end here.


Just to be clear, the proper autotools-style default would be `${CHOST}-pkg-config` first :). But if no other part of the build system does that play, nevermind.


run_cpu, it shall run for only 100 steps, and will print out the novelty of each frame and the epsilon(well, I call it rho to match the pdf) of each step.


Inconsistency spotted! Either use `%s --cflags %s`, or `+` everywhere.


There's no reason to do this check here.  `number` will *always* eventually be `0`, so you can put `translated_string.join` outside the loop.  If there are "extra zeroes" at the end, you'll be appending things like `"IV" * 0`, which evaluates to the empty string `""`.  So no harm done.


The timeout valoes are hard coded in the config.pp

Carsten BIERMANN
System Engineer
+49 228 763 776 43 | +49 160 97 24 15 16
cbiermann@talend.com | www.talend.com<https://www.talend.com/>
Talend Germany GmbH | Servatiusstrasse 53 - 53175 Bonn - Germany
Amtsgericht Bonn HRB 15336
Geschäftsführer: Emmanuel Samson, Thomas Tuchscherer, Harald Weimer
Steuer-Nr. 205/5745/1189
Ust-ID DE254821324
[http://info.talend.com/rs/talend/images/talend-logo.png]
[Facebook]<http://www.facebook.com/talend>[Twitter]<http://twitter.com/talend>[LinkedIn]<http://www.linkedin.com/groups?home=&gid=812977&trk=anet_ug_hm>

From: Stefan Baryakov [mailto:notifications@github.com]
Sent: Montag, 20. April 2015 19:47
To: cbiermann/puppet-nginx
Subject: Re: [puppet-nginx] TIPAAS_4460: Increase timeouts (1dce630)

Why fork to change default params values? Maybe it's better to change them in hiera on case by case basis.

—
Reply to this email directly or view it on GitHub<https://github.com/cbiermann/puppet-nginx/commit/1dce630afb181334bba44c8fb89cd8c52c882eb0#commitcomment-10808399>.


updated README


update the README. `bower i` is still in there...


Instead of subtracting `arabic * roman_times` from `number`, you can do

```ruby
roman_times = number / arabic
number = number % arabic
```

Do you see why?  Ruby even has a handy method [Numeric#divmod](http://ruby-doc.org/core-2.2.0/Numeric.html#method-i-divmod) that returns a `(quotient, remainder)` pair as a 2-element `Array`.


That's okay, I prefer to avoid doing manual work like manually adding myself to a list of contributors when it's trivial to automatically figure it out from the git history. Thanks though!


1995? what


Filename needs to have a .md extension @sunmeng0421


Then a link like ![Image of commercial trucks](http://www.penskeusedtrucks.com/img/img_callout_centers.jpg) should work!


Fixes #2


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2003


Варианта два.
Можно сделать локальную конфигурацию для js также как для php (на каждом сервере, где запускается проект, будет свой файл)
Проще: можно использовать относительный путь ```/api/v1/user```


done


Just a description of the error. Before, if you went to COS/S1995/ it would show a legitimate page with an empty graph. Tyler and I fixed some of the logic so it 404s now as it should.

On Tue, Apr 21, 2015 at 12:33 AM, Advait Chauhan <notifications@github.com>
wrote:

> 1995? what
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/thoff13/CurveIt/commit/8a3e437dc0da6b6120d8ee9e70a7101d38a275f9#commitcomment-10816804


@marutosi why is this needed? `install-openldap` seems like it should belong under `install`, right?


can you revisit all the naming conventions for packages,classes ?


`app/views/faq_groups/building.md`
The link doesn't have to go through a google search :)

`app/views/faq_groups/launching.md`
fixed link the final link showing was https://assembly.com/help/assembly.com/start which result in 404

`app/views/faq_groups/platform.md`
fixed link the final link showing was https://assembly.com/help/www.assembly.com/chat/general which result in 404

`app/views/guides_groups/platform.md`
fixed typos in 2 links


https://github.com/upadhyayap/PramatiTraining/blob/master/WebCrawler/src/com/pramati/imaginea/utilities/CrawlerConstants.java even the naming conventions for static variables


travis swithes container whether sudo needs or not.
"bundle install" does not need "sudo".

http://docs.travis-ci.com/user/workers/container-based-infrastructure/


It is post merge. So it is either from something in the range stuff or something from the old code that is now being used differently.


Right, don't change Sky9x code here, as it is not related to ADC!


Not in the driver!


Looks good, but I wonder now if it is a good idea to include such a change in 'master', while we are almost ready for 'next'.


Test!Test


Yes but I'm getting error

      CtorsStreamTests.cs(18,26): error CS1061: 'System.IO.MemoryStream' does not
      contain a definition for 'Close' and no extension method 'Close' accepting a first
      argument of type 'System.IO.MemoryStream' could be found (are you missing a
      using directive or an assembly reference?)


It's still blocking players from being moved from other dimensions to your dimension.

> "You cannot move players that are in another Dimension"

Why did you add this?


Yep, basically inherited from previous patch.


You haven't added the `warwars.xml` file that it links to.


88737258-EPW-Ruby-Access (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (2179 of 2201 tests passed)

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


@marutosi ruby-ldap/ruby-net-ldap shouldn't be on containers, but you're saying marutosi/ruby-net-ldap is defaulting to containers?

Also, to clarify, I don't think we should split up the `./script/install-openldap` call into `before_install`: it's an installation step. Does `before_install` or `install` have hidden `sudo` behavior that I'm not aware of?


No, this is correct


fixed


致乐网络 网站源码
www.chilee.xyz


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1166 of 1168 tests passed)

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


this


No clue why, Twitter didn't pull down, and read as deleted - fixed in second commit


This is great! Thanks for doing this.

One request: can we please get rid of vcs-version.sh for a pure Python solution?  I've found shell scripts to be less portable than Python, in particular, sed is problematic on OSX vs. Unix.  I'm not against leaving it as a shell script, but we should really avoid using sed and it's probably easier to hack up that functionality in Python than bash.


typo


Downloader


typo yo -> you


a


can simplify this logic.


Arreglando los enlaces de los Front end frameworks


Nothing jumps out at me, sorry.


@dnahodil A stricter shouldUpdate function created. It also uses the values within the form.


The form didn't validate anymore, it says:

"Select a valid choice. 5 is not one of the available choices."

I look into this in more detail.


New error page looks great and much improved.

@mnapoli Could you help solve this issue http://forum.piwik.org/read.php?9,126057 ?


didn't see u already fixed two of those leaks.


@Sobak I totally missed this comment, we usually reply to the mails to php-webmaster@ rather then here.

docs.php.net used PhD from git master -- normal mirrors/rsync does not.

The rsync server runs the latest released versions and I don't think there has been a release yet since this change was added.


Errors should show something similar to line 32 specially on login services


How about something like

```Java
Block elevator = GameRegistry.findItem("OpenBlocks", "elevator");

if(selectedBlock == EnderIO.blockTravelPlatform || (elevator != null && selectedBlock == elevator)) {
```

To make it where the Travel Anchor can interact with OpenBlocks elevators above and below it.


This was the big one. Memory usage still increases veeeeerrrry slowly so this is probably still another but it may be livable.


Yea there's still something else though. Look at the delete function in
range set. That may be another but when I tried to delete the old range it
was crashing.
On Apr 21, 2015 1:05 AM, "Akiira" <notifications@github.com> wrote:

> didn't see u already fixed two of those leaks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Akiira/Comp-524-program/commit/f851388e3aba56abf149aac58bb0c39566d68da7#commitcomment-10817030>
> .
>


other than the error handling looks good to me, next time we'll try to avoid scriptlets


this should probably be in a service but it's ok for now


fixes #593


Oh ok. Missed this because it used to use the set input parameters by
reference
On Apr 21, 2015 1:10 AM, "Akiira" <notifications@github.com> wrote:

> This was the big one. Memory usage still increases veeeeerrrry slowly so
> this is probably still another but it may be livable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Akiira/Comp-524-program/commit/52e249a7aeb4198027314a234814899f72d3bbc7#commitcomment-10817069>
> .
>


パケットのデータファイルは、features/open_flow13/ に置いてください。あとディレクトリ名が open_flow13/ 以下なので、.raw のファイル名に 13 は入れなくてOKです。


Oh... I will change password. Thanks


...well the question still remains on why it's hardcoded groups


Agreed.  Hangover from the days use_prob was a kwarg.


is the Parser used anywhere else, if not we can remove the class


Sim,
Em 20/04/2015 16:09, "Lucas Almeida" <notifications@github.com> escreveu:

> Sim
> Em 20/04/2015 14:25, "Alexsandro Santos" <notifications@github.com>
> escreveu:
>
> > Boa tarde! Encontro confirmado para quarta-feira a tarde?
> >
> > --
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/lasilva/cursoAndroid/commit/17c8b810df83e91406e1205b1fea0dec26b83f56#commitcomment-10808018
> >
> > .
> >
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/lasilva/cursoAndroid/commit/17c8b810df83e91406e1205b1fea0dec26b83f56#commitcomment-10809936>
> .
>


記述式の作問なのに選択肢？


上のコメントと同じく


88737258-EPW-Ruby-Access (jasondegraw) - 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)


Gracias. Me parece que al final tendré que pasarme a gulp.


why its redundant?


This should be called _shouldUpdate (with an underscore) to make it as a 'private' method because it is only called form inside this class.


This logo is so much better than the one I threw together, it makes mine look like a joke :stuck_out_tongue_winking_eye: Great work.


The possible values are 660 (the current value, where the owner is 'users_uid|root:abrt') and 640 (the former value where the owner was 'abrt:users_gid").


D'oh, so hopefully the right abstraction is `validate_transform_move_and_notify `.


Can I suggest this would be easier to read if the part after the OR was all on one line.

i.e.:
```javascript
return (!this._operatorIsBetween() ||
    this._operatorIsBetween() && this._hasSecondValue() && this._hasFirstValue());
```
I think this would make it more obvious that it's the first thing OR all of the second things.


While you're at it: Shouldn't the lowest values simply g_advancedsettings.m_videoTimeSeekForward/Backward and the highest to g_advancedsettings.m_videoTimeSeekForwardBig/BackwardBig  ? I think they are unused now.


@sindresorhus @julien-f there is no breaking changes in API, but:

 * all requests will have `Connection: close` header instead of `Connection: keep-alive` (in Node 0.10, in 0.12 and higher nothing will change).
 * got will now ignore `http.Agent.defaultMaxSockets` value and always use `Infinity`


Make a comment in the test please. MSDN says it exists, we might follow up offline with MemoryStream.


With the above proposed line change, we can revert this line back to using no arguments.


APIを使うときはAppControllerのapi_restメソッドを使ってください。
json_decodeされた内容が返って来るようになってます。


The body of this block should be on a new line. We only use inline block for small functions.


Yeah, sorry for the pylint contortions.  I'll see if upgrading to the hg version of astroid makes it deal better with numpy and if so go the standard route.


This test is unnecessary now as we check within shouldUpdate(). So this line can be killed-off.


In what cases would we expect this to be false (ie. the selected operator object to be null/undefined)?


Good one


タグにコメントできるのか？


A nice amount of code has been removed/tidied here which is good to see. :+1:


@1999 просто смотрю, как тяжело переехать. Этот код я не до конца мигрировал, но в принципе - возможно и сложностей не возникает.


This line should be marked as code in shell syntax


Acho que com isso podemos brincar um pouco e ver as possibilidades para o projeto.


OK, `extension-pkg-whitelist=numpy` in my pylintrc fixes this. Ahhh, more standard-looking code on the way. Sorry about that. I use the binary state change between zero/one pylint error via text editor integration.  Which sometimes leads to contortions.


Hi,

I can't reproduce #168 with cs_CS.UTF-8. I can't see why the value should be changed to 1,0 within the form automagically and I can't see why floatformat:0 should work. In fact then the values are not valid choices, e.g. 1 vs 1.0.

I'm gonna remove floatformat:0 at all. If this is really a matter one have to reopen it and come up with the working solution.

Thanks
Kai


Yes, it is the commit before https://github.com/CoWinkKeyDinkInc/OCN-Maps/commit/9fc0b0120207a856e1eff43189d433d8ebc19ce5


我会翻译成：将 network 的数据以 HAR 格式拷贝。@SunLn 你怎么看？


Sounds harmless, minor is probably fine.


Updated the commit message and made the change clearer.


Not a big problem. I was a bit careless there.


Isn't this the same as `buffer.Length`? Seems like unnecessary repetition.


Yeah. I simplified the duplicated code a bit also later


fix #32


HAR 文件是一个 JSON 数据。感觉稍有点不通顺。HAR 文件格式是 JSON，内容为XXX  ？


> lexical-binding must be set in the first line.

https://www.gnu.org/software/emacs/manual/html_node/elisp/Using-Lexical-Binding.html


Actually we want to quickly enable wiziq plugin for one of our customers using hosted canvas instance.Wouldn't making this plugin a rails engine instead will make it dependent upon your release of conversion of `vendor/plugins` to `gems/plugins` thereby causing delay in availability to our end user.As of now we need to make it available to our client asap.Which path do you suggest?A trivial old rails vendor plugin or a rails engine?And how long will it take to make it available for hosted instances once the code is reintegrated into `master`?


yes, looks good to me.


Yes i can review it.

Thank you
Anand upadhyay
On 21-Apr-2015 10:11 am, "Purna Boyapati" <notifications@github.com> wrote:

> can you revisit all the naming conventions for packages,classes ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/upadhyayap/PramatiTraining/commit/01e06c4fd42cfadaf0594269677b47f1b85a81f7#commitcomment-10816847>
> .
>


The modules won't load unless a modules folder exists inside the sim folder.


I don't know if it is the case here but in the past I looked at the buffer size that's necessary to get good write file performance and it was 32k-64k.


Looking at this now, this line is basically "shouldUpdate". What about if we move these calls to validate into the _shouldUpdate method and then just have the one conditional here. Make sense?


Refer: #7


potential cause of #33


Thanks for help


Maybe use a URL normalization library, to get a consistent result across all URLs? (and not just checking the end slash)


下面的截图展示了一个 JSON 格式数据的响应 ？ @SunLn 你看下怎么翻译比较好？


感觉 preview 这类面板上的名词不用翻译比较好，翻译成 资源预览  反而需要去找下是哪个面板。


True, thank you, I accidentally referenced an old commit in the email I sent you. The pull request #950 has a commit with `g_settings_privatereports` set to `true` (https://github.com/abrt/abrt/commit/9218946bb29805dbe2d085b9533b598644af097b). I apologize for misleading you.


> you're saying marutosi/ruby-net-ldap is defaulting to containers?

Yes, because there is no "sudo" at .travis.yml.

>  I don't think we should split up ...

http://docs.travis-ci.com/user/build-configuration/#Build-Lifecycle

> Run before_install commands
> Use this to prepare the system to install prerequisites or dependencies


Here's the line/paragraph that mentions how to see a list of all admins.


@aembler You beat me to the punch :wink: Sometimes I'd prefer that the Earth was flat: I'm start hating time zones :wink:


Test


May be in your tools, you can trap the SIG{__WARN__} or SIG{__DIE__} , and catch pattern,
That is ugly.

May be ask on IRC to mst what is opinion on this. Generally speaking it give good advice.


revertは一個前の状態にパブリッシュされるのか。


Indeed it does =)


hello Abishek


Ok, makes sense. Will push it in the next commit.


k xa


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Unused


good


@oodavid not sure this one does what you want, and it's also an extremely alpha/experimental feature! :)

This one is currently restricted to the latest browser APIs (service workers), and is meant to replace the html5 offline cache manifest:
https://github.com/gameclosure/devkit-core/blob/refactor-simulator-connection/src/clientapi/browser/cache.js


You changed this line and that change was unnecessary. That was a blank line and you've added two spaces here.


The container.javaHome may not be necessary since it's in a container environment which means the location of JDK is fixed and gerrit can find the right javaHome by itself.


I think if the pilot's throttle is above zero the vehicle won't auto-disarm even though the motors aren't spinning because E-stop/interlock is engaged.  So the vehicle could be sitting on the ground for a long time, without the motors spinning but still never disarm?


I had mistakenly thought `GC.plugins` was unused, having added it once and never used it for anything.  I'm not sure it's a good API for building a native bridge though -- seems like that should live in a `NATIVE.events.register` or something.


@maximilien please help to review this commit to implement the story of [92185380](https://www.pivotaltracker.com/n/projects/1299870/stories/92185380). Thanks!


I think this is an unintended change.  It'll mess up the EKF altitude.


-1
This command is implemented in FirefoxDriver only at the moment. So this commit broke all the other drivers. Let's start from the remote end.


Could we get away without creating the output_armed_zero_throttle method and instead just use output_min?  I think they do the same thing.


A lockfile only keeps information of NuGet packages. We don't need to invalidate a lockfile when there are some changes in `frameworkAssemblies`.

So looks like we don't need to have framework assemblies in the verification section of a lockfile.


S/b namespaced to strange.extensions.promise.impl I presume?


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@marutosi from the same link:

> Run install commands
> Use this to **install** any prerequisites or **dependencies** necessary to run your build

In this case, OpenLDAP must be installed because it is a dependency for those integration tests.


This is nice.  Alternatively we could move the numbers to be beside each item in the list.
i.e.
AUXSW_SET_HOVER = 1,
AUXSW_FLIP = 2, etc
adding the extra "=" beside each item ensures future programmers don' mistakenly add items into the middle of the list.


This is the README file.


wrong methodname called on buttonclick


U R Welcom.

kkthx
JH

On Mon, Apr 20, 2015 at 1:48 PM, Tyler Lofgren <notifications@github.com>
wrote:

> Thanks for ruining the formatting. /s
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tlofgren/CentralLibrarySystem/commit/c4aa3933bad04b09629b36fffa857aeeec66f667#commitcomment-10810696>
> .
>


Ah, this is nice to move the aux options into their own flags structure.


* 主選單要有連結可以直接連到三個主要功能的列表頁


oop


OK. I close #198.


Please, just add the comment about the value returned :-)
(Or I can do it just before merging once Travis is OK with this)


I feel like this command `\e` should never be sent through the `pgexecute.run()` routine and it should be treated like `\c` or `use`.

I'm going to cover the reason in a separate comment.


It's mapping a function on a list of items, that's the very original purpose of ``map()``. And for example, in Python 3 ``map()`` doesn't produce a list. I just don't like the the extra space there.


c'est pas un peu overkill pour wget, grep et 2 substring ? ou j'ai raté une étape, il faut plusieurs pages et une session pour avoir le contenu?


Yes, but you can't use the ingame registration, because it requires a registration server that no longer exists. Register like described in the readme:

You can register the binaries by putting the registration file in the binaries folder into this folder:
Windows XP: C:\Documents and Settings\Username\TribalTrouble\
Windows Vista or newer: C:\Users\Username\TribalTrouble\
Mac OS X: Library/Application Support/TribalTrouble/
Linux: ~/.TribalTrouble/


it works? GSM, SMS/MMS, USSD, Internet?


lol


We should simply get rid of the erroneous test functions in marssceneparser.py. This is stuff that should never have been committed in the first place.


Hello,

I'm Florent BRODZIAK.

the goal of my modification was to help new comers to finalize easily your installation without any installation problems as I've encountered with my Raspberry pi 2.
I'm working on IOT department of Sopra Steria compagny www.soprasteria.com

Thanks & regards

Florent BRODZIAK
Sent from a mobile device. Excuse brevity and typos.

-------- Message d'origine --------

De : Jeffrey Dare <notifications@github.com>

Date :21/04/2015  06:24  (GMT+01:00)

A : OptionMan/iot-raspberrypi <iot-raspberrypi@noreply.github.com>

Cc : OptionMan <fbrodziak@free.fr>

Objet : Re: [iot-raspberrypi] Update README.md (01d1ba6)

Thanks for this update. Will merge this to the main stream. Also, can you please tell me your creds? :)

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


@stevengill mismatch on *s around "npm"


LG. Where is the dashboard for this data?


2 spaces, everywhere else uses 4.


Where is detach implemented?

Honza


what‘s the fucking code !


OK, then we should clean up the newlines when we fetch the title itself. The problem here is you cannot ask for a multi line format, such as the PROPERTIES block, as you're cleaning the whole generated string. I'll add a commit that cleans the title line.


Can you start a new Issue here on github for this or post to the pox-dev mailing list rather than commenting on a commit?


Maybe it'd be useful to prefix the wfProfile calls with a common prrfix so it's easily searchable.


I think that onPacket, onAttached and onDetached can be removed.

Honza


So, this comment can go away too.

Honza


产生10为长度的密码


Jolla lipstick[973]: [W] QDesktopServices::openUrl:198 - static bool QDesktopServices::openUrl(const QUrl&): The platform plugin does not support services.


How about formulating this more positive? E.g. something like "We're particularly interested in styles for journals and published style guides and may accept some others as well (see the [criteria for inclusion](https://github.com/citation-style-language/styles/wiki/Criteria-for-Inclusion))."


Nice to hear! I'm also doing some re-organization/refactoring to allow for custom key bindings an better extensions. I'd love to hear about your changes, I'm very open to any kind of contributions at the moment. FreeNode is good, I'm known as 'wavi' there. Let's have a chat when you have time.


We should probably log an event in the dataflash logs if the e-stop is invoked.


Could be replaced with `#isSubmitted()`


this should be done for greciliae too right?


Cool, thanks for contribution. I'll need to get some time to test it before I can integrate your change.


The documentation is unclear to me here. You mention "along with all setup dependencies of all packages". But then you just call `libraryRoots`, and there's no further talk about setup dependencies. What's going on?


Hi there.
Arch user here, and installing slowmovideo from git.
I do believe that this commit broke the install paths of slowmoUI and slomoFlowEdit.

They now end up in /usr/, rather than /usr/bin/

Can you confirm that this was intentional?


Allegedly, `$this.toggleClass('active')` should do this. However, I've had issues in the past with it, so maybe this is for the best.


le premier appel ouvre la session. Pas réussi à l'avoir en direct


quand à dire que c'est overkill par rapport à écrire et compiler un programme en C, je te laisse compter le nombre de lignes ;)


This lock is unnecessary. `ConditionalWeakTable` is already safe for operations on multiple threads.


C'est le secret hash de Github.


No need for `type="text/javascript"`. There's kids in Africa starving for bandwidth, and you're wasting it on superfluous attributes.


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (641 of 642 tests passed)

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


Here is the method from Hangfire.Pro.Redis:

```csharp
        public override StateData GetStateData(string jobId)
        {
            if (jobId == null) throw new ArgumentNullException("jobId");

            var entries = Redis.GetAllEntriesFromHash(
                _storage.GetKey(String.Format("job:{0}:state", jobId)));

            if (entries.Count == 0) return null;

            var stateData = new Dictionary<string, string>(entries);
            stateData.Remove("State");
            stateData.Remove("Reason");

            return new StateData
            {
                Name = entries["State"],
                Reason = entries.ContainsKey("Reason") ? entries["Reason"] : null,
                Data = stateData
            };
        }
```


@ramanpreetnara SMH.


Please update this to use the new `mcontext` function.

This way, if a Node is passed that isn't in the graph there won't be an error (just return an empty list instead).


Hmm, do I understand this correctly?

You are computing all "roots" of the dependency graph by checking which vertices have an indegree of 0. Then subsequently, you compute the dependency closure of these roots in order to check for consistency.

What happens if the graph contains dependency cycles? These cycles will not necessarily end up in any subplan for the consistency check, right? So there's potentially not going to be a consistency check for cycles, correct? Now, all this is fine, I think, because cycles are also a problem and will hopefully be detected by a different sanity check. Nevertheless, I think this is subtle enough that it should be included in the documentation.


Without a race condition:

```csharp
StrongBox<object> box = GeneratedHeaderPresentCheck.GetOrCreateValue(tree);
box.Value = autoGenerated ? GeneratedHeaderPresent : GeneratedHeaderAbsent;
```


Are these pointing to qa site on purpose?


That doesn't work as this repo contains the actual source only, not the Ember CLI Addon.

I'll release 0.8.0.beta-2 later.


Why did you changed default value from 30 minutes?


I don't feel strongly about this, but is this actually going to deter anyone who we don't want to deter?


looks like all the jdk8 tests are failing


In principle ok.


I'm not sure if I'm a fan of  adding all kinds of files of all possible IDEs and editors. I will however add
*~
to ignore temp files in general (this should work for Emacs, gedit...).
Also, it seems that someone has at some point cloned the phobos.wiki repository into their phobos folder, which is not a good idea as in general git repositories should be kept separate.

So I'll be cleaning the file, thanks for pointing that out.


Normally I'd agree, however there's 56 states listed above. This throws off the math below on #L27.


Este cambio elimina mis modificaciones hechas para mostrar el icono de glosario en algunos ckeditors.
Ver:

https://github.com/chamilo/chamilo-lms/commit/36071b3dfd19ed9e9912eaf26a6a554d73fd6fd4#diff-a3c7e24e0e185801535ed61def93be4a

$config['extraPlugins'] = $this->getPluginsToString();


Should we make this a variable and change the "echo Testing using 8 processes" above so that it will stay in sync?


This patch is wrong (thread_set_exceptions instead of mutex lock,...).


What is the motivation for having empty arrays?
/ping #242


/ping @fmma @robeverest


fixed


Looks good. In the future, perhaps refactor the DSL into a module separate from the actual tests?


@yangyubo Thanks a lot! :+1:


If you made this change to your SickRage you broke it.  The code works as it should..

The following codeblock is actually dead code and affects nothing in the source further on:

```python
        if not plex_server_token:
            token = sickbeard.PLEX_SERVER_TOKEN
```

As the variable token is overwritten completely in the "username / password" section when not used with a token.

All you're doing with this code is stomping all over the function call variable plex_server_token and corrupting it for the next IF statement checking to see if it had something in it from the call to update_library


This commit seems to break the NPM installation, as NPM looks in the dependencies key too, the github:xxxx/xxxx urls are invalid for NPM.


Can you make this (and `SymbolFlags.None`) consistent with the others by using similar hex notation?


You don't use `data-legal` for anything.


Falta la documentación PHPdoc de todas las nuevas funciones agregadas.


AFAIK the "from phobos import *" is only necessary in case we have submodules, which we don't. In case I'm wrong, we should still put in the commit message why Eclipse is being cranky about this. ;)
I'll fix the imports in defs.py.


:-) Gracias!


do not treat function blocks as block scoped containers


Looks ok to me (but I have less confidence about this part of the code in general).


That's not your name!


I don't think you should use 'is' like that. It can lead to problems depending on how you manipulate last_move.

>>> a='A'
>>> a+a is 'AA'
False

See: http://stackoverflow.com/a/2987975


This can be written as `.void`.


As far as I can see, there are plenty of unused variable definitions here that have nothing to do with the problem. I'll sort this out and check which line really causes the problem, I have a suspicion already.


Well, I was a bit bored, and I thought I'd give it a go ... always open to further modification of course ... I was thinking of maybe turning the bottom part of the '2' into a belt, with the 'crop' rolling on it, and the 'p' falling off at the end ...


这个项目设置有必要每次都commit嘛?


Ha. LOL. Monads is an inappropriate conversation. Nowhere to leave a comment on your post so I figured I'd leave one on the commit.  :dart:


Tydligen kunde man efter senaste ändring med scenen logga in oavsett vad man skriver. Räcker att bara lägga in en boolean eller liknande i LogInMenuController. Fungerar bra då. Istället för "if (!"".equals(storedName))" i handleChangeToMenu kan det vara userCorrectCheck = true. Sedan sätta till true i  if (rs.next()) statementet  i handleButtonLogin().


Get rid of the `return`s


Modifique:
               en UtGen la variable NL que estaba mal posicionada
               En Generar Funcion, agregarlos saltos de los return
               En llamado a funcion, falta cargar los argumentos en las variables


@thet if we do not load the testing profile any longer it should be removed alltogether.


Ok.


You don't need another selector for `h2`, you already have on in the line above. You can add definitions to it :smile:


typo : initilize  -> initialize


always have a meaningful commit message


Please try to avoid commiting `config/config.json` unless you are making a new variable.


Indent the sample further.


is this required & used anywhere?


It would be nice to remove illegal content


Why pointer to config, and not just struct?


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


Use 4 spaces from the base indentation of the example.


Yes, in a way. I refactored it while searching for the cause of the problem with the request token not being validated correctly upon GET requests. Eventually, it was not the listener causing the problem.


Extra IdentityRole1Id


block-scoped


Nice one! I love this game.


Extra foreignkey


Should this back off? Reminds of the day when our hg server was down. And stayed down because all our automation retried so hard that it couldn't  get back up.


extra foreign key


Why not do the same for CharacterComponent?
I'm still having trouble grasping why RenderComponent is causing crashes, and not CharacterComponent.
See `server/lib/Map.cpp`, where all components are initialised.


try to always use {} with if to make the code clean..


Default color is `#777`, but that's also the color of `banned`. So, when you uncheck one thing, it's impossible to tell what state is unchecked and what's banned.

Maybe make banned red, and some of the middle ones (decriminalized, medical) more yellow? Then it'll look "Bob Marley"-esque as opposed to "Easter-themed Web Safe Colors"-esque.


+_+ Good...


Как определяем, когда тестировать сочетания?


Otherwise 'x' would not appear to be a redeclaration of a block scoped local in the following example:


Shouldn't we send errors to STDERR? https://google-styleguide.googlecode.com/svn/trunk/shell.xml#STDOUT_vs_STDERR


@andyjmaclean per the [Ruby style guide](https://github.com/bbatsov/ruby-style-guide#collections), please "use the Ruby 1.9 hash literal syntax when your hash keys are symbols."

The preferred syntax has been applied to this file in commit 8561e8950a04e80f2724e5c9d3f8e3c737654b75.


Done.


Cleaned up a bunch.


*would* be placed - "will" makes it seem like this is the current behavior.

Remove the hyphen.


Ok.


notes on offending lines.


Fixed #20


Any idea when this is going to be added to the oficial releases?


Выше нигде работа с выходными значениями не фигурировала. Стоит в описании метода об этом упомянуть.


I just followed `// TODO: check the database instead?` and made the server look the database instead of the filesystem for a file with the same name, I did not take into account the fact that there could be files on-disk that were not present into the database (so the problem might still be present).
However, the only reason I can think for an "on-disk file" not to be in the database is that it was uploaded/created manually bypassing the uploader. In this case, maybe it would be better to add some sort of service that monitors changes in the filesystem and adds new files to the database automatically.


@lkolbly @sprakes How is this API for processes (an abstraction many contributors work). What do you guys think? Anything to add. I am not sure how to handle forking yet. Duplicate the vas and the PCB structures? Let me know if yall see any problems to this.


All the cool kids haved moved on to [Haas Unica](http://www.dailydot.com/geek/neue-haas-unica-font-helvetica-release/).


try to remove the +1 and -1 if possible... seems not necessary.


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


Why did you remove the comment?


Oh no, there was a time before the database. That's why this remained a /`/ TODO` for so long.


Why not set `selectedChip` to `-1` ? Is there a special meaning to `null` ?


Avisst sörrö!


'locals' field


'locals'


"will be need to place some child in" -> "will need to hold some child node"


As jason mentioned, it's inaccurate :)  I need to come up with a better one explaining how enums actually work.


`bigDC` can never *not* be `true`. Maybe just remove this `if`, and add a comment?


Not a big deal but you don't have to stringify the types. `(t/assert= Integer (type (rem 5 3)))` should also work.


The OpenID Connect spec mandates that the redirect_uri in the token request matches the redirect_uri from the authentication request. See http://openid.net/specs/openid-connect-core-1_0.html#TokenRequest

Not checking it opens for at least a handful vulnerabilities.


Addressed.


lol


Why on ~~this night~~ the end of this sentence do you trail with one space, but on all other sentences you trail with two?


why is 'block' in single quotes?


ok. Let's start it!


This was changed in ac8fd01b52262a1dc9d8cf6a221f6d5083d1743d.


Yikes.


Understood, then the problem is still present.
I shall make a script to add all of these files to the database, maybe I'll add it to the control panel to make it easier to use.


because this well new field for the object


You don't move to Sponge Project ? ='(
Your plugin was amazing, if you have some time, go visit me on play.eredrim.fr. We have few game build with it.


Commit reviewed

I agree with the approach here. The only thing I would add is a convention based approach to adding the properties file. Picking up properties would thus go as follows:

Look for property file defined in sitehierarchy
if no file is defined, look for a file named properties.json


instead of modifying the current queue, you can also use the inner list that you built..


*favicon


You will have a conflict with your other PR


I think I actually have one written for the most part in my `Code` folder. I'll take a looksie


Awesome, thanks


hhmmm something still feels incorrect.
1. Why Super Novice +2000 HP has been put in base HP/SP table? Aegis doesn't do that and even iRO wiki says it as bonus, added in additive (fixed amount) HP bonus.
2. Why the +2000 is started from 150?
3. Just remembered, I only check and separated Novice X Sunov HP for re, not pre yet.


resourceOperation/get.json 等项目中没有


@jdoliner Will you take a look at lines 277-280? Can't decide how to fix this. Maybe I'm using transaction IDs incorrectly?


Есть нюанс, какими должны быть эти данные, чтобы для их тестирования именно эта техника подходила лучше всего?


Sorry for the noise, but is your intention to actually check in with folks following through your commits? I noticed two commits back you were like "anybody here". If so, at least I do periodically check the repo and the commits :)


Саму по себе неоднородность классов эквивалентности можно отнести к минусам техники?


可以再续传一下么，谢啦


TeamCity xBIM :: XbimExchange :: Build [Build 242](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1457&buildTypeId=XBIM_XbimExchange_Build) is now running


... You are aware that git has this killer functionality called "revert"?


If you're going to have comment blocks between declarations, forget alignment; use one space before the `=`


no need to meld the words together


The removal of the () apparently sets the $count to one. If we need this, probably better to remove the count completely.


TeamCity xBIM :: XbimExchange :: Build [Build 242](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1457&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:24


and potentially block-scoped container


block-scoped container


jaja, es otra cosa si


typo ?


See also here: http://stackoverflow.com/questions/19455285/conflict-cygwin-with-winsock-h
>  ... its not advised to use CygWin with Winsock


TeamCity xBIM :: XbimExchange :: Build [Build 243](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1458&buildTypeId=XBIM_XbimExchange_Build) is now running


Thanks :+1:


Philipp only disabled OAuth functionality in the GUI. This is not really a "revert".
Anyways, the fix will come today.


split into two lines


TeamCity xBIM :: XbimExchange :: Build [Build 243](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1458&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Compilation error: Xbim.Exchanger\Xbim.Exchanger.csproj (new) Build time: 00:00:54


Great


:sparkles:


Much nicer


function-container


Adding the widger profiler would mean we have to switch to jKs widget handler, instead of basecontent, I think. I don't think we have any expensive LuaUI (and I can't imagine scene ed does), so this is probably not necessary.


@nfeske: please add to staging


Ainda não esta pronto :P


lower-case the 'f' in 'Function'


okay agree


With [EmpiricalRisks.jl](https://github.com/lindahua/EmpiricalRisks.jl) ready, I am now reviving the development of this package.


No, the function will only write 2 bytes. It uses sizeof( object ) to determine how many bytes to write.

Are you sure you weren't using a an `unsinged long` which is 4 bytes.


changed it a bit but most of it is the same, still has a lot of problems


> Please try to avoid commiting config/config.json unless you are making a new variable.

How about not tracking it in git then?


别黑了。。。


Ok good job


> Why not do the same for CharacterComponent?

Because it doesn't crash.


Hmm why we need the dagger?


Вопросы по Domain Testing:
1. Почему Domain Testing связывают с тестированием многомерных областей определения?
2. В случае двумерной области определения, какие виды ошибок можно найти с использованием этой техники?


这个模板能添加节点事件吗？


avec ou sans les libs ;) ? mais ok, on(tu) installera python-request et
comme ça ça ira, je patcherai le script pour l'associer à la course.

2015-04-21 8:26 GMT+02:00 paparazzia <notifications@github.com>:

> quand à dire que c'est overkill par rapport à écrire et compiler un
> programme en C, je te laisse compter le nombre de lignes ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/v-l-m/vlm/commit/0fbe6863c913e19ff4e613a74cc5fbc7992ca6e9#commitcomment-10817729>
> .
>


the CTT are very peaky, for instance they wait for a second longer as the announced lifetime and re-use the seesion id... i fear we will also have to check the expire date of every session/sc on every login.... just wanted to warn


Thanks. Fixed in 1.5.0.5, 1.4.4.5.


Действительно ли Exploratory testing никогда не планируется заранее? И если планируется, то как его можно запланировать?
В каких случаях Exploratory testing не требует использования документации в каком-либо виде?


@indutny Yes, I've found them on git repository. We need not to apply private patches from 1.0.2b.
I also found an another backported patch in https://github.com/openssl/openssl/commit/47daa155a31b0a54ce09ad2ed4d55fad74096dab . I wonder if this is needed in 1.0.2a since there is no loop for a cert chain in checking a issuer in 1.0.2a where there is one in master as https://github.com/openssl/openssl/blob/master/crypto/x509/x509_vfy.c#L538-L544 .  But I might have missed something.


So does this mean, that youtube-dl will always download the best video available when there is `ffmpeg` available on the system? Like 1080p/60fps from Youtube with no further format specification? Before it was stuck at 1080p/30fps without stating the format as `bestvideo+bestaudio`.


Patch as a whole looks good, but could you leave a note about possible optimization just above this call to emasculate_binary?  Otherwise I fear the issue will be forgotten...


В чем именно состоит непрозрачность исследовательского тестирования?


That doesn't seem right. Why does it scroll to begin with?


My suggestion was more to revert the merging of the OAuth bits in the first place and then reapply the long-url fix. If it's not sustained for a longer time, I'm okay with it, but this feels almost as awkward as the time I put a zipped repo in the git tree:P


Is this intentionally left like this? And the test enabled?


This is an excellent idea! I am so glad I did not work on an alternative "let's mangle `MSYS_TTY_HANDLES` when spawning" patch over the weekend!


LGTM, especially like ```@TupleConstructor```


Meh. Strings are empty by default.


Let's move this block below, we need not have all the `#include`s at the beginning. It makes more sense to have them in the place where they are needed, if they are needed.


Change to `idx = -knit - 1`. It's not an unary `-1`.


(e.g. Restkit 0.10.3) should be capitalised (RestKit). The rest of these changes do a great job of articulating the pros and cons, nice work 👍


I am of two minds here. While `FDEV` is definitely also an internal detail, `FPIPE` is really only needed when detecting MSys2 tty pipes, so I could imagine that moving the definition into the `#ifdef`-guarded part would make sense. But it's not a *strong* preference, so if you want to keep it next to the `FDEV` definition, that would be fine with me, too. Just food for thought.


Moin,

why should we trust someone who is too lazy to explain when something like https://translate.google.de/
exists?

Carsten


Good to know. I haven't CygWin installed and only tested compilation with MinGW so far.


The steps on line 1026 onwards need to be reviewed and updated so that they always overwrite - in particular, if the dictionary members are empty or missing, the result is that the corresponding values change to "unset" or the default value.


Hmm. This name is informed by an implementation detail. I would rather have it state the purpose. I.e.

```c
#ifdef DETECT_MSYS2_TTYS
```

This would clearly inform the reader what this guard is all about.

Objections?


Like the `USE_NTDLL` above, `msystty_init()` strikes me as wanting to be renamed to `detect_msys2_tty(int fd)`.


Why a response status 300?


Problems description
====
Singleton in DB access is not a good idea. When two requests arrive at a very close time calling executeSql(string) for instance, there will be an exception if the code execute like this:
1: Request 1 arrives first and have already finished invoking comm.ExecuteNonQuery() and about to close the connection.
2: Request 2 arrives, and just have executed the code before comm.ExecuteNonQuery(), now is about to invoke comm.ExecuteNonQuery().
3: Request 1 now invokes closeConnection(), however Request 2 haven't called comm.ExecuteNonQuery().
4: Reqeust 2 invokes comm.ExecuteNonQuery() but get an exception telling that the connection is not opened.

Conclusion
====
1. DO NOT use singleton when dealing with DB.
2. Add a lock if you insist to do so.


It's necessary to merge this IMMEDIATELY, as people could have broken databases due to power loss/crash with journal_mode = OFF

I agree this is a good solution for now to be safe, but I think we shouldn't have to workaround to do tests.
I have a PR to do for this, but I would like to also use your changes here as a base , so please merge this ASAP.

After it is fixed right, I think the `testdb=True` workaround should be taken out. I have done a PRAGMA locking_mode = EXCLUSIVE and it has fixed the tests, AND it is the safest way to journal, AND it improves performance. I just need to test the normal operation of SR using this directive, because I am not 100% certain we don't try to make more than 1 connection to any given DB at a time, which EXCLUSIVE does not allow.

I also rewrote tearDown_test_db(), much more complete and efficiently, will PR after this is merged.


Very nice, it will make our code clearer. Maybe we can provide fake implementation of GoogleApiClientWrapper for tests?


While this is technically correct, it is just repeating what the two lines below do, but leaves the occasional reader still without the most crucial information. Let's write this instead:

```c
/* mark as a device so that isatty() determines the tty correctly */
```


vector layers -> vector layer


The original statement caused nothing wrong actually. But when I printed `os.path.dirname(__file__)` I got nothing but a blank line. Therefore, I add `os.path.abspath` for ensuring loading json correctly.


Please remember tu put new line at the end of files. More info: http://stackoverflow.com/questions/729692/why-should-files-end-with-a-newline


@danieleguido, didn't you mess with the files' tabulations per chance?


Попробуй сам сформулировать плюсы, минусы и область применения. Может быть, еще что-то добавишь.


no, it was not intentional ! I don't know why but, I can't make both OSX and Linux install correctly  at the same time !
can you test this patch :

--- a/src/slowmoVideo/slowmoFlowEdit/CMakeLists.txt
+++ b/src/slowmoVideo/slowmoFlowEdit/CMakeLists.txt
@@-62,7 +62,8 @@ target_link_libraries(slowmoFlowEdit sVgui sVflow sVvis)

         #install(TARGETS slowmoUI DESTINATION ".")
         #install(TARGETS slowmoFlowEdit DESTINATION ${DEST})
-install(TARGETS slowmoFlowEdit DESTINATION ".")
+#install(TARGETS slowmoFlowEdit DESTINATION ".")
+install(TARGETS slowmoFlowEdit DESTINATION ${DEST})

 if (Qt5Core_FOUND)
         include(DeployQt5) # 2.8.7 or later
@@-75,6 +76,6 @@ if (APPLE)
 elseif(WIN32)
        install_qt_executable(slowmoFlowEdit.exe "" ""  )
 else()
-       install_qt_executable(slowmoFlowEdit "" ""  )
+#       install_qt_executable(slowmoFlowEdit "" ""  )
 endif()

--- a/src/slowmoVideo/slowmoUI/CMakeLists.txt
+++ b/src/slowmoVideo/slowmoUI/CMakeLists.txt
@@-166,7 +166,8 @@ Categories=Qt;AudioVideo;Video;\n")
 )
 endif()

-install(TARGETS slowmoUI DESTINATION ".")
+#install(TARGETS slowmoUI DESTINATION ".")
+install(TARGETS slowmoUI DESTINATION ${DEST})
 if (Qt5Core_FOUND)
     include(DeployQt5) # 2.8.7 or later
 else()
@@-180,6 +181,6 @@ if (APPLE)
 elseif(WIN32)
        install_qt_executable(slowmoUI.exe "${PLUGINS}"  )
 else()
-       install_qt_executable(slowmoUI "${PLUGINS}"  )
+#       install_qt_executable(slowmoUI "${PLUGINS}" ${DEST} )
 endif()


I've only copied the same introduction than appears in the Overview section, from the official website, but feel free to propose a new one :-)


Here the serialization method may not support handling boxed type. So is it not necessary for current schema?


I guess I would prefer explicit guards `#ifdef DETECT_MSYS2_TTYS` here. It is the same amount of lines added (i.e. two), but makes it clearer what is done here.


otra cosa


Yep


Я бы вынес эту кнопку в отдельный класс


I agree. Would you be ready to add that event to delite/Container so I can leverage it?


I meant when using btrfs in root partition


Yes if you look at my current branch (which overrides this commit), this is probably already removed.


@shigeki I'd stay close with the 1.0.2b anyway, but I do see your argument. :) Let's update to it once it'll be released, I think it should happen soon.


![Image text](https://raw.githubusercontent.com/mengmanzbh/img-folder/master/d.png)


There is just one puzzle left... I remember distinctly that I had to `#undef` my `isatty()` override in `winansi.c` because some output (which I unfortunately do not remember now) was redirected to the (hidden) console by mistake unless I did so.

If I am not mistaken, this problem does not occur during the test suite, but only when using Git interactively.


Avant de faire la méthode personnaliserNiveau(...), il faut rajouter une nouvelle classe niveauPersonel qui hérite de niveau. ET la tu peut appeleé le constructeur de niveauPersonel


That's what I naturally tried but it `undefined` as well...


@isaacg1 What's the difference between `.r` and `.<` or `.>` on `<seq><int>`?


you might want to check `rgb()` too, no?


Traffic Data


because we have to check the size of the list to go to the end


Не ясно, как от диаграмм переходить к тест-кейсам.


This whole block could be a filter instead i think, would be cleaner.


We don't indent relative to the expression start, so it actually was correct before.


> There is just one puzzle left... I remember distinctly that I had to #undef my isatty() override in winansi.c because some output (which I unfortunately do not remember now) was redirected to the (hidden) console by mistake unless I did so.

My guess is that [this part](https://github.com/kblees/git/commit/7d57fb45117b335aa58cfc1436557a366f934546#diff-5a07b657681304b199c3bbacce334efdR628) fixes that issue. Well done!


ok


@gau1991 Mention issue no in your commit message.

So people who reported issue are aware of we are working on that issue


...and issue #7


tо use this class


I don't think this is right.  The alignment checks in bcopy.c check for word alignedness and for CHERI we must copy one capability at a time when copying the capability sections.  I'll tweak the asm version to handle different alignments depending on capability size.


He is just checking that in the previous prop there is an hexa (#) color at the same place as the rgba in the current prop.
I think it's doing the job pretty well for now.

If we want to improve that we might want to check something more accurate than the exact same position  (eg: `background: #000; background: linear-gradient(rgba...`).


Hmm, doesn't the executable have to be wrapped. Or what exactly uses `PYENCHANT_LIBRARY_PATH`


@indutny Let's wait it until 1.0.2b. I'm just wondering if we need to apply the new backported patch to iojs immediately.


@MiteshShah +1

Issue number should always be part of commit message.

Ref: https://help.github.com/articles/closing-issues-via-commit-messages/


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1144 of 1151 tests passed)

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


`biberexe` should be  `biber`, not `makeindex`.


TEST2.csv is a small dataset to use for testing.
trip_data_january.csv is the full (HUGE) data set, which contains about 1 million entries.


Hard to tell. I feel that in your formulation you actually have to change the "and" into a "but" from a language/logical point, which then sounds even more deterrent.


C-Programmers habbits are hard to forget ;)


Eftersom den här delen endast kör när vi ska uppdatera databasen (kanske en gång i veckan) så vore det inte ett stort slöseri med resurser att skapa en ny klient. Webservern bör ha en klient som den återanvänder.

Detta gäller så länge det inte finns en annan anledning än att spara resurser för att inte skapa en ny klient...


I did this by tutorial, without this line, the line is not working:
BufferedReader in = new BufferedReader(new FileReader(file.getAbsoluteFile()));


Hi,

This line is withdrawn here to https://github.com/PrestaShop/PrestaShop/pull/2899/files#diff-fbb038f4b8b267c6eddb7098259b8f54R220

Regards


EBIN :---DD


Why change the early check `fd == 1 || fd == 2`? Not that it *should* matter all that often, but it is better to perform these computationally cheap tests *first* (as it is done before your patch), so that the more expensive tests can be avoided when they are not even necessary.

So while the code still tests the same things after this patch, in many cases it performs unnecessary tests, too.

Since the `fd` and the `hconsole` tests are the cheap ones, I imagine that the code could be improved thusly:

```c
if (((fd == 1 && hconsole1) || (fd == 2 && hconsole2))
    && isatty(fd) && GetFileType(hnd) == FILE_TYPE_PIPE)
  return (fd == 1) ? hconsole1 : hconsole2;
```


Could you change the commit message to make it a fixup to the commit it reverts? That way it can serve as a reminder to drop this patch with the next merging rebase.


Well that's a framework problem then…


About those dirs, I follow the instructions from the original Makefile, to make a debian package by the same purposes as release a tarball from that Makefile. In case those directories are needed, where do they be installed? And maybe we should to review the original Makefile, because the "tarball/release" tag doesn't include them.

About the init scripts and a config file, I'm agreed. This was only a proof of concept for a first releaseable debian package, with minor requirements to get a fully functional one. So we should work to improve the package, by providing more contents, like yours. It would be also desirable to write a man page, and maybe a default logging directory.

While writing this comment, I think that maybe it would be a good idea to discuss about which should be the best default configuration file in #prometheus/irc or the mailing list.


The len > 3 is not really necessary.


Why use an unstable version in our master? (Around gnome, odd minor numbers denote testing version.)


I just quickly updated the SQL (pardon the indentation fuckups, I'm not paying enough attention right now to bother), I wrote this like 2 years ago but it still seems to run and runs _incredibly_ fast.  Configuration is through `config.json`.  Just `npm install` and `node ./master.js` and it'll do the things.

I guess it's up to @nokonoko to run it

https://github.com/NuckChorris/pomf-scanner


実にすばらしい!!!


I was going for easy toggling, but good point - I should just remove the conditionals and the class


Crap, yep, didn't check it, I just needed this commit running :D. I am really sorry, my fault.


oh this is pretty, thanks


Can't it be just `if (missingTextures.length == 0) return;`


so every new class you add will be prefixed with 'New'? :P please remove it :)


@shigeki doesn't seem to be necessary. I'll ask Matt about the purpose of this patch.


so why not static does not work?


Yes, We will write issue no onwards :)


Include a NULL check instead. The result should be an empty collection in the ember model.


This could actually be added to the while condition: && count++ < 1000.


This check does nothing. If `$relation->getRelatedModels()` returns an empty array, nothing in the foreach will be executed.


@indutny Thanks!


/cc @mattbasta


fffffffffffff


use the initializer list, please.


方法命名方式不统一，O-c一般采用驼峰


@Subjunk Not only slower but also degrades quality:-)
BTW. MEncoder counts final resolution from res.width+AR+mod so for 1920x??? MEncoder adds black bars up to 1920/16*9=1080 for any mod lower 16. For 16 final resolution should be 1920x1088. Similar with 1280x??? where up to mod 16 is OK.
I can't believe that similar filter is missing in FFmpeg :-(


@ejm not @em


Pushar du ändringen eller ändrar vi manuellt?


This file should not have execute permission. Fixed.


КурсовойПроект/project.txt

- давай так: "такой метод подготовки данных для тестирования модели системы или продукта"


Yes I get this if pop-ups are blocked. If I allow popups then this does not come up.
But as I said, I think the OAuth needs these 3 "Drive API", "Google Picker API", "Google+ API"


oo


Wouldn't it be cleaner to pass the configuration as a constructor parameter for simulations ?

As far as I can tell, 2.2.0 is already a breaking change with regards to configuration because of the removal of the configuration property on `GatlingConfiguration`. People will have to edit and recompile their simulations anyways, making them add a constructor is not much more difficult than referencing `Predef.configuration` instead of `GatlingConfiguration.configuration`

cheers


a


更改类名，单词错误。


Better get the height also into a var first.


Aren't standard edids 256 Byte size?


There's a colon missing at the end of this line.


КурсовойПроект/project.txt

- Стоит добавить информацию о том, каким образом строятся комбинации для пар (математическое обоснование подхода)
- "связывание по ключевым позициям" - скорее нет. Здесь используется математический метод, какой?


@noahmalmed
Can you improve this test code so it checks inside the byte array for hash values and side?


what about to use Objects.isNull or Objects.nonNull ? More Functional code == better codef


DEBUG=true
GOOGLE_CLIENT_ID=xxxxxxxxxxxx.apps.googleusercontent.com


Some explanations here would be good why +=2 and why +=3 that's not obvious when one does not know how an EDID info is parsed.


TODO:
Provide more params for the onReceive callbacks for both Server's and Client's.


What?! Autoboxing?


Another hidden autoboxing ?!


Again, this file should not have execute permission.


Swift或者OC端的测试用例一般使用mock的方式，可以先调研一下。


Indeed :-).


Quoting Domen Kožar (2015-04-21 09:47:14)
> Hmm, doesn't the executable have to be wrapped. Or what exactly uses
> PYENCHANT_LIBRARY_PATH
>

there are no binaries. but i see. my change will brake whatever uses pyenchant.
fix coming in 5min.


you know what? i'm just gonna change to urxvt and that'll fix it :P


Also, pretty sure you dont need the commit() at https://github.com/abeloin/SickRage/blob/81ca934af0261cc7367734ddcbc4c0298f215407/sickbeard/db.py#L64


I like this idea, but why not use enumeration?  Something like:
```cpp
enum cateogry {beverage, appetizer, soup /* .. */};
```
might be more readable and precise.


这里直接使用字符串，一旦更改，需要更改的面积太大，尽量做静态变量处理，并于DataMenger关联在一起


nice find!


The `if`s to qualify each function seem to be causing compiler errors, yet if I use `static if` blocks to the same effect, things work. Am I doing something dumb like placing the `if` in the wrong order with the `return` and `@safe` qualifiers or something?


- `npm uninstall -g gulp`

- удалил папку `node_modules`

- снес `Node.js`, поставил заново

- `npm install -g gulp`

- `npm install` в папке Dollchan-Extension-Tools

ЧЯДНТ?


Just checked it quickly and it seems to be fine, should work without problems (SQL is right, the other values have defaults they can fallback without causing errors)


这样的现实也不要完全采用字符串的形式，一旦变化，修改量很大。


Does "assertTrue(True)" do anything?


nevim, fb přidal tenhle post type, předtim to byl tušim `link`, nechce se mi to s tim dělat nic dalšího, eventy stejně nesdílíme, ne?


“提醒时间”多次出现，尽量减少重复


better than NPE :]


“网络异常”多次出现，尽量减少重复。


I haven't used `make-fn-schema` in the past, have been meaning to look into it and haven't gotten around to it.  Have you used it before?  I assume that your intent is to change the args to make it enforce the actual inputs/outputs of the callback function later?

I think that's probably a really good idea, just wanted to check and make sure I'm understanding your intent.


I would also not add IDE files, this is why I ignored them by adding them to .gitignore. Or do you mean you do not like adding them to .gitingore?


functional code is evil, also JDK 1.7


类名？随机Time？尽量不用汉语拼音吧


To be clear, I suggested disabling the visual aid of the white vertical line (like we disable the Quick Play tooltips visual aid).  I find the white line invaluable when I do Quick-Play but the line gets in my way when going up to the menus,

Perhaps the mouse pointer should be normal arrow when over the Timeline with Quick Play disabled, as it is when recording?


activities


I thought I needed some assert for the test to succeed but I just checked and it's not needed, removing.


ErrorCode集中起来的好处是什么？


Use `StringUtils::EqualsNoCase()` here as well?


And missing space between the parameters.


Not a huge deal but I think we've been doing the tk-j9 dep as a 'dev' dependency in most projects so far (though at some point it might be worth discussing whether or not we think that pattern has been useful/successful).


将下面的代码添加到你的代码库


使用这个管理类的三个步骤


helpful tip, be consistent with css naming schemas, for example use only lowercase and dashes, "collector-current-status", "collector-status", that is the most common way to name css classes and the one used in Grafana, so its best to keep it consistent :)


Fixed.


@cpcundill adding the event to the document and then filtering the callback on an element is a technique known as event delegation. It is very useful and has a number of benefits. Two really good reasons to use this as an approach here are: adding event handlers to every individual instance of an element on the page is expensive both in setup and maintaing those event handlers.

The second is that by using delegation you don't have to worry about the contents of the page changing and new elements getting added or removed. If, for example, there was a tabbed interface which loaded new content onto the page when you swapped tabs this code would work with the new tab content without having to re-initialise it.

I also don't think your tone is very helpful here. Saying "To be honest, I think this polyfill isn't too well written" isn't the kind of behaviour we appreciate or want on our repositories, it is needless critical commentary. Please try and make your comments more constructive and specific in future.


Nice catch!


I think using a local variable like
'''
material_name = MaterialObject['Object'].Material['General_name']
'''
could improve readability (for use in line 516 and 519 as well)


深入思考


> yeah I don't question calling it, I question not setting
> SYMBOLS_COLUMN_TAG=found yet using found in that call. Reading a bit
> deeper seem to suggest it *is* fine though, but I maybe setting found=tag
> in this case would make things clearer.
>
I understand now, yeah, found=tag might really be clearer.

> This said, while I understand that removing the whole gtk_tree_store_set()
> call is probably good for performances, we need to understand that:
>
>    - the tag should really be *completely* equivalent (e.g. good
>    comparison function) as we use it for everything starting at the tree
>    (goto, etc.) -- well, in practice it probably doesn't need to be *that*
>    good currently, but better not get weird surprises in the future;
>
> The function compares all the fields so it should be the case already.

>
>    - this might mean higher memory usage, because the tree store would
>    keep the old tag around, while the tagmanager keeps the new one. Basically,
>    instead of 2 refs to the same tag, we get 2 tags;
>    - shouldn't be a problem, but a collateral is that the tag *pointer*
>    is *not* in any of tagmanager's lists.
>
> Good point, this didn't come to my mind. But I think in the tag tree we
always need to compare values instead of pointers anyway so it shouldn't
matter. And I wouldn't worry about the memory usage.


I'd use a different formatting here. Instead of "`what to do`" I'd use "_what to do_". You want to emphasise on it, don't you? And in most other places you use `this kind of formatting` for code or keywords.


Could you use "or" here instead of 2 separate "if"?


hello


PAPRIKA!


use variables when feasible for the max-width, min-width, it will make it a lot easier to update the breakpoints (and see what breakpoints exists)


line 230 reg?


КурсовойПроект/project.txt

"Если значения выборки не связаны -- лучше провести отдельно для каждого значения свой тест."

а = [true, false]
b =  [1, 2, 3]

Перебор каждого значения по отдельности дает наборы (~ любое значение)
true ~
false ~
~ 1
~ 2
~ 3

Если соединить в пары:
true  1
false 2
~      3

Получаем три кейса вместо пяти.


Leaking x509_str here


Lots of typos. For instance, it's **règle** and not **régle**.


@MartinNowak, would this suffer from the `moveEmplace` issue?