@reFX-Mike Since I opened this PR I think I have to explain the change.

Yes, you're right that many times `const` make code clearer because it says the variable is not changed later. However in this case all the code we have is a short one-liner and it's trivial to see that it doesn't change the variable (and the variable itself is discarded immediately afterwards because the function ends). On the other hand having a function parameter declared as `const char` makes it unclear that it was the actual intent and the parameter wasn't meant to be `const char*` - this takes a moment or so of extra though and extra though is not what we value in third party code, is it?

So we have a next to zero benefit of explaining that the variable isn't changed in the tiny one-liner that follows and a slight risk of making the code harder to read. In this specific case (very short code) I'd say that getting rid of `const` makes code clearer.

If you have an example of how dropping `const` like this prevents code optimization I'll be happy to dig into that - that will be a day when I'll learn a lot.


Why not just remove it?


In few mpconfigport.h's we have:

~~~~
#define realloc                                     gc_realloc
~~~~

i.e. it's assumed that gc_realloc's signature matches standard realloc. This is now broken.


its all fine


but we are using local file. should we use as fallback?


Yup. Thanks. That works well.


Could you check hasattr for keys and `__getitem__` and then do dict conversion?


The question is what would that return value be.

One of the reasons that I like this approach is that it doesn't necessitate a return value for "incorrect usage" that users would have to deal with, because return values have a meaning specific to the function being defined. I don't like any specific return value being allocated to this.

Isn't it enough to document this callback as "after this callback returns, anything can happen - including crashing"?


Got it. setup.py fails when Cython is missing. So the failure only happens when installing from git (with pip or with setup.py):

```
jay@JayMBA:~/Desktop$ which python
/usr/local/bin/python
jay@JayMBA:~/Desktop$ python --version
Python 2.7.10
jay@JayMBA:~/Desktop$ virtualenv venv
New python executable in venv/bin/python2.7
Not overwriting existing python script venv/bin/python (you must use venv/bin/python2.7)
Installing setuptools, pip...done.
jay@JayMBA:~/Desktop$ source venv/bin/activate
(venv)jay@JayMBA:~/Desktop$ pip freeze
You are using pip version 6.0.6, however version 7.1.0 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
greenlet==0.4.7
(venv)jay@JayMBA:~/Desktop$ deactivate
jay@JayMBA:~/Desktop$ rm -rf venv/
jay@JayMBA:~/Desktop$
jay@JayMBA:~/Desktop$ which python
/usr/local/bin/python
jay@JayMBA:~/Desktop$ python --version
Python 2.7.10
jay@JayMBA:~/Desktop$ virtualenv venv
New python executable in venv/bin/python2.7
Also creating executable in venv/bin/python
Installing setuptools, pip...done.
jay@JayMBA:~/Desktop$ source venv/bin/activate
(venv)jay@JayMBA:~/Desktop$ pip freeze
You are using pip version 6.0.6, however version 7.1.0 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
(venv)jay@JayMBA:~/Desktop$ pip install --verbose git+git://github.com/gevent/gevent.git@v1.1b1#egg=gevent >gevent-v1.1b1-pip.log
```

Output: https://gist.github.com/parasyte/0c2a0304b643c466bcc3#file-gevent-v1-1b1-pip-log

```
jay@JayMBA:~/Desktop$ which python
/usr/local/bin/python
jay@JayMBA:~/Desktop$ python --version
Python 2.7.10
jay@JayMBA:~/Desktop$ git clone git://github.com/gevent/gevent.git
Cloning into 'gevent'...
remote: Counting objects: 16837, done.
remote: Compressing objects: 100% (66/66), done.
remote: Total 16837 (delta 26), reused 0 (delta 0), pack-reused 16771
Receiving objects: 100% (16837/16837), 7.85 MiB | 3.87 MiB/s, done.
Resolving deltas: 100% (10342/10342), done.
Checking connectivity... done.
jay@JayMBA:~/Desktop$ cd gevent
jay@JayMBA:~/Desktop/gevent (master)$ virtualenv venv
New python executable in venv/bin/python2.7
Also creating executable in venv/bin/python
Installing setuptools, pip...done.
jay@JayMBA:~/Desktop/gevent (master)$ source venv/bin/activate
(venv)jay@JayMBA:~/Desktop/gevent (master)$ git checkout v1.1b1
Note: checking out 'v1.1b1'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 5e9a521... Versioning [skip ci]
(venv)jay@JayMBA:~/Desktop/gevent ((v1.1b1))$ python setup.py install >../gevent-v1.1b1-git.log 2>&1
```

Output: https://gist.github.com/parasyte/0c2a0304b643c466bcc3#file-gevent-v1-1b1-git-log


Dod you forget to remove the SSH stuff in comments?


I am a nice comment


I searched the man page, `--help` and source but unfortunately did not find how to run Mixmaster's tests. The simplest test I could make was sending a dummy message (`mixmaster --dummy`) with `subprocess.check_output`, but it always return `0` for both success and failure. It was adding some delay too.

This fix currently checks for paths Mixmaster usually uses for both compiled and installed (with the package manager) versions:

| | Compiled | Installed |
| --- | --- | --- |
| Binary | `~/Mix/mixmaster` | `mixmaster` |
| Configs | `~/Mix/mix.cfg` | `~/.Mix/mix.cfg` |

nymphemeral will first check if the paths configured on `nymphemeral.cfg` work, and after that, check these default values. It recognizes the installation by calling `--version` on the binary and checking for the existence of the config file. If it is some derivative of `Mixmaster 3` and the config file is found, it assumes Mixmaster is installed and working. Finally it searches for the *mix chain* to be displayed on the GUI, but will not prevent Mixmaster to be used if it's not found.

Further checks can be done by parsing `mix.cfg` entirely. So far, it seems to fix #8 without adding delay on startup and does not require the user to run tests separately.


nice


Maybe?  Though-- for example, I then couldn't use the callback to construct a test harness that verifies that all the expected argument checks are performed on all the functions. Rather than allocating a return, it could return whatever the callback returns?


It seems to me that in Ruby it is idiomatic to not use parenthesis in conditionals (unless you absolutely have to to ensure precedence of some operators over other operators)... I'm no Ruby expert but that's what I've seen.


Should be:

```
<button data-toggle="aside" class="link-like" role="button" type="button">options panel</button>
```


If there is a PT version of this Wikipedia article, let's link to it.


Or perhaps it can guarantee it won't crash, but not that the return value of the API call has any meaning (it could be implemented as just return 0, I guess).


Need to track this in the DB:

```
GroupEmailThread(id, project_id, group_id, email, date)

message_id = hash(id + email) # or something that doesnt use just the ID
```

The In-Reply-To and Reference would point to the given message_id, and when its first created, that email would gain the Message-Id. Followup messages can be whatever.


Looks good. Will commit this when sourceforge is up.


Fix #1


Also some kind of API that doesnt restrict emails to just group threads so we could expand later might be good:

EmailMessage(thread=GroupEmailThread.for_group(group))

GroupEmailThread = def for_group(cls, group): return EmailThread(message_id=..., reference=..., in_reply_to=...)


Hey @eduardolundgren, first results varying only these two parameters (remembering that we want to minimize the sum of distances): https://gist.github.com/cirocosta/fc511e96b765696e5984 (it lacks teaking also with `minSize` and `maxSize`).


This change brings me this **Notice (8): Undefined index: _ids [CORE/src/View/Form/EntityContext.php, line 226]** when the belongsToMany relationship is empty.


That's how you do that?!?!  Now I need to fix my shitty fix for isFakereddit


It's not clear to me why this is abandoning the trailers supplied by the caller.


No need, the news should be removed from main repo after this, and possibly the CHANGES too should maybe be changed to just include a link to CHANGEAS.md in the other repo.

We don;t want to worry about maintaining 2 copies of the same file.


Should this be sizeof (vertices)?  Interesting that it works anyway.


If the request fails, fallback can just be a markdown link to the github.io page that failed, so they can click it and open it in their browser


Possibly a dumb question -- why can't it reset the stream with `DEADLINE_EXCEEDED`? Is it because the only API to abort is a `cancel()` method? If that is the case, would it make sense to push deadline handling deeper down the stack, so it could call a method on the concrete implementation of the stream (vs. only being able to use methods on the public interface)?


@UniversalMediaServer/developers I think that it could be also set to .mpg beacuse the mime type for MPEG-PS/MPEG-TS is the same video/mpeg. What do you think?


Numbers will now be expressed in letters and dashes.


Do either of these need to be thread-safe? If not, why not?


\o/ thanks for sharing! worked to compile on raspberry 2 armv7. i'm going to add a bit of google bait here. The error i received before this branch was:

```
shmat: Invalid argument
terminate called after throwing an instance of 'std::runtime_error'
what(): circular_buffer: shmat()
```


In the broken commit `model.Job.user_id == None` was also changed to `model.Job.user_id is None` and similarly elsewhere. Should this be changed to `model.Job.user_id == null()` ?


nit: this wants to be `@link` instead of `@code` (and maybe the same goes for the new reference added in `@throws`?)


Today I learned that sizeof is an operator, not a function, and brackets are only needed when referencing types. The brackets are redundant if you are referencing a variable by name. :smiley:


When no GPL header exists for a script, then you bring in above header without any prior copyrights or authors(so, remove Rod). And since you are making changes to the file, then add yourself to copyright and author as you have done).


Would it be possible to use opengl to draw the border (4 quads) after the glTexSubImage2D operation?  It would save the hassle of saving the pixels etc.


Better just to use the build-in `Array.isArray` method.


Why a skip-list map instead of a concurrent hash map?


Looks good. I'll bring this into the codebase (I'll remove Rod from the GPL header).


agree


Maybe use weak references for values to mitigate memory leaks?


Size of Packet


Could do! BRB...


21 in the other SMSG_PET_SPELLS_MESSAGE u added is swaped which is one correct it pls.


Yup, it's expected that if you install from git you must have cython installed. I don't believe this has changed. Here's part of README.rst from 1.0.2 (1.1 and master have the same language):

> installing from github
> To install the latest development version:
>    pip install cython git+git://github.com/gevent/gevent.git#egg=gevent

This commit was intended to make a particular scenario on readthedocs easier, it wasn't designed to completely remove the need to have cython available. I only tested that one scenario on readthedocs, that's why I didn't make any documentation changes.


@Mindbreaker1
Any special testing should be left to the maintainers.  If you want to convince them properly I suggest you post your exact testing setup with full results.

Interesting note: If the used multiplier of 8 was any number larger than DEPTH_MAX  this would be a non functional patch.


I was confused because the same test using PyPy works without Cython until this commit. So I guess this commit "fixed" that particular situation.


I think that a more generic name like CPlayerProfile is better for this class. Although most information here now is saved progress of missions the player has played, there is also information about player's appearance, which is something else. And I suppose in future we may add some more data that is also not related to mission progress.


xlt()


xla()
(since this is within an html attribute, it uses xla() )


The reason I didn't do this first time round is because I was unsure how to achieve pixel-perfect accuracy when drawing the lines. But I'll have another think about it.


And so robotmain.cpp & maindialog.cpp have lost some more lines of code. Good, very good :)


The html ouput step does not happen here, so clearer to not do the html escaping here. Instead, keep these all xl() and do the escape when escape the variable below.


attr()


text()


Hmm...are you sure it was this commit? A slightly earlier commit got PyPy to start compiling _semaphore.pyx with Cython, whereas previous versions, including 1.1a2, had PyPy using it as pure-python code (the problem with that is that it was non-atomic and could be interrupted by signals---not what you want in a synchronization primitive!). Could it have been that commit?


remove strip_escape_custom(). This is already taken care of by the $sanitize_all_escapes flag.


Is pixel-perfect accuracy needed?  Nobody will be examining their retina screens under a magnifying glass to check ;)


Don't escape it until you send it to the html (so remove the text() in just the one line above).


Oops missed that one


Need to use binding on the query (use the strategy of creating a sqlParam array which you then push the variables into it that can then be used on the sql call)


Meh, I dunno. I'm just OCD :stuck_out_tongue:


done


use binding


Just a quick question. Should the csv labels be translated?


this commit is part of a pr that has also caused regressions for me, see #1447


Crushed it


These are not getting outputted to html, but instead to csv format. So would run these through the escaper. So if translate, would use xl().


Is `tokenhosts` even still used?


Now some dynamic formats get tested twice.
CircleCI only has CPU formats, no GPU, so you could have changed the first --test=0 into --test=0 --formats=cpu-dynamic to skip the dynamic formats.


attr()


I observed that you generally update the offset after you read values. However you forget to update it in 2 cases:
1. bool
2. When you break with qDebug()

Should you update in those cases too?


You use the same type for all the vars, so why union? For better readability?


I get "AttributeError: 'Select' object has no attribute 'table'" from this line when I attempt to filter on a hybrid property.

Here's a gist that reproduces the error: https://gist.github.com/pawl/6589cebdcacf9afe7e50

Relates to #949


Wouldn't translate this. Since if an error is made, it will be translated at that point. So, just need to escape with addslashes()


Um, why Java? And why in this repository? I'm rather negative towards adding Java dependencies in the repo.


Since you're saving the records as quint32 isn't it possible to allow also 28bits and 32bits here too?
Or do your bit operations below depend on the fact that the record is 24bits?
(again I am not good with bit level operations)


Ah, good question, although it should [default to false](https://github.com/jshint/jshint/blob/4c2d091b7e50ce2681ee48a104578bb969c189ae/examples/.jshintrc#L42).


The Makefile.in change affects Travis builds, not CircleCI builds as stated in the commit message.


Line 173 needs binding in query


line 191 needs xlt()


Btw, I had a very pleasant experience with using nickel.rs as a minimal frontend server in my CorTeX reimplementation (I'm spending pretty much all of my time on that now). Here is the current state:

https://github.com/dginev/rust-cortex/blob/master/examples/frontend.rs

The only reasonable justification of adding Java dependencies would be to use the Stanford NLP toolkit, but I would much rather use it as an external dep and not bundle it with LLaMaPUn (if we use it at all)


An expression will have one plaintext length. It shows in the format details


Line 231 needs text()


Line 259 needs binding


Just installed image jessie-rpi2. Need a password to login as root. couldn't find it anywhere. Is it a big secret?


Does your mother know you are so diao like this？


Good point! It may not have been exactly this commit. I've been installing the tags, rather than HEAD. You're right, bisect points to https://github.com/gevent/gevent/commit/f053bc97940b69a35941bd2df221601b6b4bc786

In which case, I'm seeing expected behavior from README!

Cheers.


There are 2 files changed in the commit. One for each CI


change to text()


Place each of the strings in xl() to translate them.
It's always better to get the string in the xl() than a variable so they then get sent to the trasnlation spreadsheet.


This was probably unintended.


done


I have observed that you treat all queried IPs as IPv6, since you convert IPv4->IPv6.
So what will happen if the loaded db is IPv4 only?


Who cares if some formats get run twice. We have had instances where bugs in duna crashed other formats, but only if Dina ran. I think both test runs are fully valid


Is the script compatible with all Python versions? Can we get away with just ./schema_db.py ?


Ok, the problem is that the command doesn't work for me. Can you fix it?


line 766 binding


added a delete function


Can we explain the format of the files, i.e. what a 'program' is?


line 860 binding
line 868 binding


line 946 binding


line 990 binding


line 1008 binding


Not a big deal, but the code would be more readable in GitHub if wrapped, for instance 80-columns.


Ok


line 1243 attr()


I would prefer this color scheme to be called "default" rather than "kakoune"


line 1293 attr()


Is setting all this infrastructure up really beneficial? I mean how much are you actually using it? Maybe mapping by hand doesn't look that pretty but considering the amount of beans and properties in this project I think that the code that would actually map things would take up less space than all this boilerplate.


Phoneline at home is still down.. But I managed to commit these using my mobile phone. :D


line 1300 text()


Note however that JS and CSS files required from HTML are downloaded directly by the client.
They work fine, at least in my browser.


Shouldn't you check for `ISO_CODE_NA` or `--`? Also why search for `'!'`?


done


It looks much nicer with argparse, thanks!


Review is completed.


Sujets de la piscine de C de Juillet 2015.


Thank you for the comment. I'll check that.

is_array() do an "instanceof Array".

2015-07-18 23:56 GMT+02:00 Arthur Verschaeve <notifications@github.com>:

> Better just to use the build-in Array.isArray method.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vmorgulys/crabble/commit/b123534ed4cceab35dd48515b36e6dbc9265a1fd#commitcomment-12239918>
> .
>


Hmm, I saw that the old code had `'!'` but I don't know why....


You could always have all your events in one class now?


How does this work? Say I wanted to build electron against a different version of io.js, how would I go about that?


I could, but it's just personal preference to have them in separate files. It just seems a bit cleaner to me.


Does this have any downside?
On Jul 18, 2015 9:53 PM, "Underground78" <notifications@github.com> wrote:

> I guess those Draw... functions are too big to be considered useful
> candidates for inlining.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kasper93/mpc-hc/commit/9a64c7c62367b5e7237754a9400fdafd4b1321ad#commitcomment-12239427>
> .
>


Ah, okay.


Personally I prefer having this way. Although it might be too much and if
we switch to SSE2 we'll need to revert it.
On Jul 18, 2015 9:41 PM, "Underground78" <notifications@github.com> wrote:

> Are you really sure it impacts performance that much? I mean one extra
> conditional check compared to the whole processing should be negligible.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kasper93/mpc-hc/commit/571213ea465a78cf254b150dc834bd757c2617f9#commitcomment-12239356>
> .
>


because it doesn't cost us anything to keep it there if one has already installed numpy.


Closes #1.


Looks good.


Please, no feeds frequencies in this file.

Also if you are going to add Ka frequencies these need to be handled as a different position just as Ku and C-band are otherwise it will be impossible to access both from the same tuner.


This is a markdown file


@asterite pr sent https://github.com/manastech/crystal/pull/1004
@bjmllr ah yeah that makes sense, thanks!


It's more of a wire protocol issue. The server really needs to use HTTP/2's `RST_STREAM` to cancel, as otherwise we'd have to at least complete any partially-sent message and that could be prevented by flow control. With `RST_STREAM`, the server can release all its resources immediately.

However, `RST_STREAM` can only communicate an HTTP/2 error code, and can't carry additional data (unlike `GO_AWAY`). The best error code seems to be pretty obviously `CANCEL` which will turn into gRPC's `CANCELLED`. But we won't be able to have any debugging data other than "the server cancelled."

The in-memory transport is simply emulating the same restricted communication to behave more like "the real thing."

We can push deadline handling lower in the stack. Transport/stream APIs used here is more of an "internal" API that we could extend. However, since the deadline _did_ expire, it is well within this code's right to deliver a `DEADLINE_EXCEEDED`. As I say in the comment, this is safe to do regardless of the Status code; we're just choosing to pick out a single case for efficiency.

This discussion topic is very new, as in I think we talked about sending `RST_STREAM` for server-side deadline for the first time on Wednesday. I wouldn't have messed with this yet except that it bit me :)


Problem of ember-selectize is due to older version in ember-cli-materialize
will be fixing soon


I see :flushed: Thanks for showing me that file, I had no idea.


## This is a markdown file


don't know :-) but i don't think so. need debuging.


Will people see this code? Maybe just do a `actLikeDoingSomeHeavyWork(...)` method so they don't get used to seeing Thread.sleep :)


Please take this out in strings.xml Similarly for others.


dimens!


tu as [générateurs] deux fois, le premier est [classes]


Putting `resizeColumnToContents(PeerListDelegate::COUNTRY);` after showcolumn() solves @ngosang's problem.
(the column is shown with size 0 otherwise :unamused:)


good :)


Rod should look at this script to ensure goes along with current status quo
@sunsetsystems


I would keep a better name, '"spotify_steamer"


It was necessary for me


![win_20150509_172438](https://cloud.githubusercontent.com/assets/13398119/8763835/ca9871ea-2d6d-11e5-9613-f24fbfbe4440.JPG)
step in two some new


@cpritam - This test appears to be failing on the Linux systems. Your gold file contains no stress in the xx direction but the Linux system is showing some noise in that field ~1e-08 so it's failing. You may have to make a custom comparison file or verify that you have initialized all fields in this need code.


ok, fixed in 5dc123042e


Thanks Frederick Zipp :-)


You should have taken a style out for the buttons.


So, based on the discussion above, if the server did exceed its deadline and send RST_STREAM, there won't be any trailers. But if it was for some other reason, there could indeed be trailers.

I did consider not replacing trailers, but it seemed unsafe. The application could very reasonably be combining the status code with data from the trailers, and so not replacing the trailers gives an inconsistent view of what the server sent. Since this is really a race, it seems fine to act as if the deadline handling happened just a moment before we received the status/trailers, which means using DEADLINE_EXCEEDED and empty trailers.

Would a comment along the lines of "replace trailers to prevent mixing sources of status and trailers" help?


salam


@today-not-someday Please have a look.


Timeout is OK to use generally, sometimes using `?` is fine, and then you need to have a timeout :)


If you want to make a serious claim that "it is "better",
please provide your test results, time control, version used, bench positions used or opening book,
and so on.


what is this?


Is QRDA stuff supposed to be here?


please use colors.xml


You have commons-io as a transitive dependency from jMCLaunchLib, [use it!](https://commons.apache.org/proper/commons-io/apidocs/org/apache/commons/io/IOUtils.html#toString%28java.net.URL%29)


ditto


Your call On translating the csv headers. I would guess that other languages would need to out put the information


@ccx-testerbot runtests master


use echo xls() without the e parameter. (instead of xl() )


I don't think so. The `break` statement forces control flow out of the directive-parsing `while` loop, so the code you have referenced is not executed in this case.


Hello! I was one of the reviewers for the project and I just couldn't resist the urge to leave a comment. I really, really LOVE your code. Your code is so easy to follow. I like how you didn't hard-code the path to the file containing the data. Your code is definitely a shining example for me. Thanks so much for sharing. :)


Супер!!! Спасибо!!!


Наздоровье ;-)


changed to xl() from xlt() for header translation


👍

but i would prefer BbCode (bb = abbrev, code = new word)


No, because ClientCall interface does not imply thread-safe. The application is required to coordinate if it uses it between multiple threads. Part of the reason it is not thread-safe is to be able to have illegal states; if it was meant to be used from multiple threads we wouldn't want calls to "randomly" throw an exception because of a race with another thread.


I see, you're right, its hard to follow indentation on github..


Too bad you're not running bundler-audit here, or you'd know this gem has a bad stack overflow vuln ;)


Thank you for the request. It has been added to the queue (id: 35). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/view.php?id=35) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Much better to keep the string together for translation. so
if
Automated Measure Calculations (AMC) - 2014 Stage 1
else
Automated Measure Calculations (AMC) - 2014 Stage 2


As above, better to keep full string:
if
Automated Measure Calculations (AMC) - 2014 - Stage 1
else
Automated Measure Calculations (AMC) - 2014 - Stage 2


done


depends which style you like here (feel free to ignore :wink:), my fav is:

```
get {
  path("instrument"/ Segment) { id =>
    // ...
  }
}
```


yup :)


Yes! Great!


Remove code instead of commenting it out. This comment also applies in all other places that you are commenting out code.


There are incompatibility issues stemming from this new approach when using Gradle 2.5 and latest IntelliJ IDEA:
* the new syntax is flagged as incorrect in IntelliJ
![image](https://cloud.githubusercontent.com/assets/3229134/8763848/0007a39c-2d77-11e5-90a5-172673a05ba6.png)
* an error shows up when I clone my project on a different computer and try to setup/import the project using its Gradle wrapper
![image](https://cloud.githubusercontent.com/assets/3229134/8763864/d4f92f1c-2d77-11e5-81bc-97645e8445af.png)


I don't know why the fuck this is here.


I see.  This is troubling, as there are three public forks of my repository already, so if I rewrite my history and rebase on to Adam's tree with Henry's patches, their repositories will no longer be in-sync with mine.  On the other hand, if I rebase those changes on to mine, I don't know who has forked/tracked Adam's repository.

I'm inclined to rebase Adam and Henry's changes onto mine, since mine is [older](https://lists.hcoop.net/pipermail/nestedvm/2013-August/000150.html).  What do you think?


Perhaps not worth adding, but… method overriding is another way to consume events, particularly view events (`viewDidAppear`, etc).


(attr($final_colspan) - 3) this is how I did this one


I considered both maps, and went with the skip list because it was lock-free and because I happened to type it first :). I would be very fine with a concurrent hash map or maybe even a `Collections.synchronizedMap()`.

Servers are required to be shut down, so it seems that such a memory leak would find a bug, which is a bit of what this transport is supposed to do :-). There are threads associated with the Server if you aren't using a direct executor, so that would be another reason to shut down. However, I was already thinking about ways to do a registry-less and maybe even throw-away instances that you don't have to bother closing. That applies to some other comments, so I'll comment there.


Cool :-)


Thanks a lot!


Is cqm stuff supposed to be here?


Тут мы включить по-умолчанию `wxwidgets` не можем, потому что у меня не собирается (wxwidgets 3.0). Проблема, судя по всему, с `libxml`


yup


is cqm and qrda stuff supposed to be herE?


♥
Looks better, eh?

![image](https://cloud.githubusercontent.com/assets/1075405/8763882/09e90020-2dab-11e5-8531-923c7646f71c.png)


*Rage quit*


response to #47


As above, better to keep full string:
if
2014 Automated Measure Calculations (AMC) Stage 1
else
2014 Automated Measure Calculations (AMC) Stage 2


done


Including in .hpp is always less good than including in .cpp


I wouldn't mess with it before the algorithm. Would attr the final product:
attr($final_colspan - 3)


what


Sounds good.


You get to decide :)


Thanks @masatake


Please follow the other described in the code guidelines for comments + use @brief


Then Translate


getNotes return a const std::vector<Pitch>& as the vector is already created for performance reasons. You might consider
getNotes()
{
static std::vector<Pitch> chordPitches = getChordPitches();
return chordPitches[id()];
}


Looks good.  People should be mindful of #25 (which this addresses).  If you need memcached during artisan, you'll want to enable it for the `cli` sapi, but disable it for `php-cgi`.


Odd bug:
The reports worked first couple times. Then the reports began timing out and getting following error:
PHP Fatal error:  Call to undefined function date_to_utc() in /var/www/openemr/library/classes/rulesets/Amc/reports/AMC_314g_1_2_19/Numerator.php on line 19


xla()


Peer Assignment 1


use echo xlt() without the e parameter


I've come to the conclusion that autowiring through constructor is more favorable in the sense that it clearly states the dependencies. When autowiring through fields, it's much harder to spot whether a bean depends on too many things. When the object has four fields, you don't care, but if you need to pass four objects to the constructor, then you start wondering whether it's actualy sane or not.


WTF. How?


You're right.  Thanks for your earlier response.  I am still experimenting and learning from your logic.  Testing both the original and as well as
weakEnemies &= ~pos.pieces(Them, PAWN)
                        & ~ei.attackedBy[Them][AT_LEAST_2]
                        & ei.splitDefendeds[Them];


be more explicit here to help future developers. Explicitly list the possible $rule_filter (ie amc_2014_stage1 etc.)


It seems like if the app is shut down for too long it doesn't ring. Also, when it rings we need a layover notification that people can tap to go to the call because as of now you wouldn't know where the call is coming from and you have to tap Minds. We have to figure something out for ios too otherwise it's going to be really hard to get people to use it reliably.


why is there a laborders and a lab-orders ?


I'll look into it


exactly. thanx!


This is an unacceptable header. Need GPL. Replace it with this header (with Ensoftek as copyright and yourself as author; remove myself and Rod from the example). And do this for all other scripts that have above header:
http://www.open-emr.org/wiki/index.php/How_to_Document_Your_Code_Properly#Page-Level_DocBlock


Hello?


I was one of the reviewers for this project. I really liked the idea of having one file perform one task. I think this allows one to easily reuse the code for a different purpose later or add additional functions later. Great job. :)


Can we give a more specific name? It's not obvious from the name that SSL is involved. Something like: get_ssl_client_info().


Mai -> mai


Maybe should be better to just add a tag `mri`, or `ruby_engine: :mri` and them exclude them via before block in `spec_helper` if jRuby is the runtime, don't you agree? :)


Is much easier to spot and it can be used by any other spec that we want skip for jRuby or other engine in the future


use sql binding on query:
``````
"AND form_encounter.encounter = ?";
$check = sqlQuery( $sql, array($patient->object['encounter']) );


I'm aware :) none of these gems are used in production so doesn't matter as much. Happy to merge a PR!


Nouvelles d'io.js?
L'actualité d'io.js?


Can't this function be used instead?
``````
Helper::checkAnyEncounter($patient, $beginDate, $endDate, $option


just seems odd to need to hard-code a pc_catid to it. Let me know if I am missing something.


Upstream `github-pages` already has a fix committed, but new version of the gem just isn't released. This doesn't affect production GitHub Pages either, as they track the gem versions separately (https://pages.github.com/versions/). Was mostly just amused by it. :)


The meta files contains information like:
Bypass sRGB sampling, required to work correctly with Linear color space.
Read enabled for the two Knob textures, required for the tinting.
It is a Unity specific project so I don't understand why you're hiding meta files to begin with. I know it's a pain to get every ones meta files sync'ed, but it's a one off pain and then it's smooth sailing.


Can't this function be used instead? (as is used in prior AMC rules)
``````
Helper::checkAnyEncounter($patient, $beginDate, $endDate, $option)
``````

just seems odd to need to hard-code a pc_catid to it. Let me know if I am missing something.


yep, c'est bien août


wow! nice :)


Is hard-coding the pc_catid a good idea? Isn't an encounter an encounter?


Would it work with utf8?

If we ever need to store non-ASCII values, utf8 is usually better.


We shouldn't need to set a non-default initial AUTO_INCREMENT.


use sql binding in the query


Include the time unit in the name to avoid mistakes, for instance timeout_sec


Where are the secure messages coming from?


It's not related to the suggestions, but it's something I remembered that made me laugh. During the Java school at C.T.Co you commented on one of the projects asking why I was using the throws clause on all the tests. YAGNI!


@duramato changes wil be here now!

also it's incomplete. check https://github.com/SiCKRAGETV/SickRage/compare/v4.0.38...v4.0.39 and check changes in changed.md - only two!


What does this field mean?


What does this field mean? (Same question below for 'sent_to' and 'taskdata'.)


What are the possible values for this field?


I've seen this a couple times. Why are you building this patArr[] array?


Did you mean "wakeup_url"?


Possible values?


Is it the maximum number of concurrent tasks? If so, name suggestion: max_concurrent_tasks


Are you sure I can do this , I am building a $where that changes it could be encounter or pid depending and there are other variables being introduced , provider,facility


In #1356: @mcarpenter, looks like this was my mistake when I tried (failed) at refactoring this code.

It should be an easy fix to change this back to nul delimiting.


ensure you have the names somewhat standardized regarding stage etc. For example the send_sum_stage2_amc is confusing.


I did not include a README file previously


We should have a primary key for the two fields together, to avoid duplicates. Then we only need to create an index for typeid (assuming we need good performance for that column).


Can we remove support for sqlite altogether?  I don't think we'd then need any of the special JSON related stuff.


让代码规范显示。


That terminfo stuff is available in any shell via `tput` and similar commands; I think those are available on just about every system; OMZ just isn't using them.

> But what's the real-world use case for installing oh-my-zsh without also installing zsh?

I can't think of any. Maybe one of the more experienced OMZ devs would know?


If chevrons-always-visible='false' then attrs.chevronsAlwaysVisible === 'false', a string, and not a boolean. So this line will always evaluate to 'ng-disabled' whenever chevrons-always-visible has a value. I've gotten tripped up on this a lot. This can be fixed with wrapping it in scope.$eval.

scope.$eval(attrs.chevronsAlwaysVisible)

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

I may fork and improve this library and can do a PR if you prefer, but only if you're interested in keeping this updated on npm (did you pull it off recently?). If not, than I can maintain it and have a version of it on npm. Thanks!


Yeah, I agree that changing the state and invalidating in onDraw is bad.  I
started out doing and invalidate right after the player 1 move.  Then doing
the AI move and invalidating again.  The problem is invalidate() doesn't
actually force a draw when you call it.  It just marks the View dirty and
it will draw at some point after the OnTouchEvent() exits.  I looked for
some event I could handle after onDraw() but I couldn't find one.

On Fri, Jul 17, 2015 at 10:35 PM supremearyal <notifications@github.com>
wrote:

> I think the sleep should go in onTouchEvent, they way it works right now
> is kinda weird. Basically a touch event is invalidating the view which
> triggers onDraw where the move is made. I think onDraw shouldn't change the
> state of the game and just worry about drawing whenever it needs to. You
> could invalidate before you make a move which will draw the latest move
> then wait and make an AI move and invalidate again (I think) for the same
> effect. Then again, the code was already kinda convoluted, I should
> probably refactor into a board class that has nothing to do with drawing
> and have the UI change the board state and the board trigger invalidates to
> redraw.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/supremearyal/TicTacToe/commit/4de32143c78ead536bb0cb7fee38976394512b0a#commitcomment-12236470>
> .
>


Totally necessary most important commit of entire project. +1


Need commit to fix spell checker.


@ccx-testerbot runtests master


An example of this would be: https://api.myjson.com/bins/39vba


Oh, I totally forgot about that. I guess you're going to have a crazy state machine regardless. There's gotta be a clean way to separate ui and state in Android though. I might investigate that...


Agreed.  We can back out the change until we find a better solution.  I
didn't look too hard.

On Fri, Jul 17, 2015 at 11:12 PM supremearyal <notifications@github.com>
wrote:

> Oh, I totally forgot about that. I guess you're going to have a crazy
> state machine regardless. There's gotta be a clean way to separate ui and
> state in Android though. I might investigate that...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/supremearyal/TicTacToe/commit/4de32143c78ead536bb0cb7fee38976394512b0a#commitcomment-12236555>
> .
>


Command acknowledged. You can find the progress and final result here: http://ccextractor.canihavesome.coffee/view.php?id=23. Please not that it could take a while before any results appear.


The typo exists in original code. Let me add one patch to correct all typos.


Thanks @balupton :)


Yeah, tackling the bluetooth stuff with a lot of state going on in a single view seemed like a recipe for disaster. So I might try separating the board stuff before I do anything else.


You can leave your changes on a branch, I'll do the same with the bluetooth stuff and we can merge on to master whenever things are ready.


So, if two equally free elevators arrive on a floor (the game still prioritizes the most free elevator - regardless of score). The game loads meeples from the floor to the elevator in an alternating fashion. This just makes it so that the first Meeple is awarded to the player with a lower score. Which also means that if there's only 1 meeple and two equally free elevators it'll go to the person with the lower score.

The idea is that two equally similar AIs should score similarly. So, we ran the demo agent against itself over the course of 35+ games and looked at the difference in score at the end. Results here: http://fouriertransformer.github.io/ElevatorsAnalysis. "Original" (~185 games) is you and your enemies elevator starting on completely random floors. "Updated" (~35 games) is them starting on the same floor (33a7d3553472aa5c50002c9272a34ffbfaff8230) and "Proposed" (~85 games) is this change.

To counteract game imbalances we're thinking you'll have to win by 10 (possibly 15) points to officially declare it a "win".

If you have some thoughts on how we came to this let us know and we'll make adjustments accordingly!


Interesting stuff: http://zserge.com/blog/android-mvx.html


Am I correct that in my example, this would loop > 50 times before getting a triangle (I took a tolerance of `0.01` and kept reducing it until I got about `0.056` which returned a triangle for me)? Either way, that's still fine as we should only trigger this check in exceptional circumstances.


Command acknowledged. You can find the progress and final result here: http://ccextractor.canihavesome.coffee/view.php?id=25. Please not that it could take a while before any results appear.


Command acknowledged. You can find the progress and final result here: http://ccextractor.canihavesome.coffee/view.php?id=26. Please not that it could take a while before any results appear.


cleanup_unit_test (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1244 of 1244 tests passed)

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


Do you have steps to reproduce that? I don't reproduce errors locally, Cython is installed if required. Note that a release tarball doesn't include a Makefile, and development has always required Cython be available.


adjusts


it will make me laugth forever sys.argv


Ah, good point.  Yeah, that would definitely break things


This is actually `arkmanager rconcmd`


  /* Trade some performance for lots of ram. At the time of writing the
     * available Lua heap went from 18248 to 34704. */
    *(.rodata*)
--------------------------------------------
Now we have only
Free IRam : 542
--------------------------------------
We can release 3.6 KB IRAM
----------------------------------------------
  /* Trade some performance for lots of ram. At the time of writing the
     * available Lua heap went from 18248 to 34704. */
    *(.rodata*)
	*libc.a:(.literal .text .literal.* .text.*)
--------------------------------
Free IRam : 4130
-------------------------------
We can release  15 KB
  /* Trade some performance for lots of ram. At the time of writing the
     * available Lua heap went from 18248 to 34704. */
    *(.rodata*)
	*libc.a:(.literal .text .literal.* .text.*)
#*libpp.a:(.literal .text .literal.* .text.*)
---------------------------------------
Free IRam : 19006
-----------------------------------------
Maybe install SDK 1.2.0 or put the constants in the IRAM?


I cloned your repo, but I don't have any additions. Thanks for taking care of it


@tractorcow Is this commit on the `develop` branch still required? I'm keen to get rid of that branch, avoid confusion with `master` or feature branches.


Uninstall Cython and the gevent install will fail. Pip does not install Cython as a dependency.

Interesting, installing 1.1a2 works on PyPy without Cython ...


Unintentional. I should have picked that up in git diff.


フォーマッタをかけるようにします。


That looks pretty cool.  I guess the problem with putting a delay between
rendering our change and rendering an AI change is the program is kind of
unidirection.  We handle events and tell the object when to reflow.  But we
don't get any information back about when the reflow is complete.  I'm
going to poke around some documentation.

On Fri, Jul 17, 2015 at 11:20 PM supremearyal <notifications@github.com>
wrote:

> Interesting stuff: http://zserge.com/blog/android-mvx.html
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/supremearyal/TicTacToe/commit/4de32143c78ead536bb0cb7fee38976394512b0a#commitcomment-12236589>
> .
>


:open_mouth: :hushed: RIP


please


The test suite completed it's run. This is a summary (full info can be found on the [status](http://ccextractor.canihavesome.coffee/view.php?id=26) page:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | 21/21 |
| _WTV | 13/13 |
| _XDS | **33/34** |
| _Options | **80/87** |
It seems there were some errors. Please check the [status](http://ccextractor.canihavesome.coffee/view.php?id=26) and [results](http://ccextractor.canihavesome.coffee/reports/26) page, and verify these.


Fix issue #56 , #54


changeは替える行為、nextは替わった後の値という位置づけで付けました。


From what I understand wkwebview has a bug reading local files and the way the media plugin currently works is it downloads the entire remote file using the file plugin and then reads it as a file.  So if local file access is not right media plugin is likely having all sorts of issues.

I'd stay away from it until the local file access problems are properly resolved in an iOS update then it becomes feasible.  It's a shame as its meant to be heaps faster!


Fixed:

https://github.com/Mei152/address-bloc/commit/3f031217eb3cfe4230e88b1044dccceefe33f729


Yeah, the way it's structured right now touches invalidate the draw which generate draws later and we're trying to get two draws within a single touch event which doesn't work. This helpful post: http://stackoverflow.com/questions/4938822/how-can-i-use-the-animation-framework-inside-the-canvas has a trick where invalidate gets called on every onDraw which basically gives you a game loop and you can basically count frames and divide by 60 to figure out when to draw/not draw giving you delay. But then again I was trying to avoid making onDraw a game loop.


inteliJで注意されたので、注意を消す為に引数を入れていました。このとき、constructを使う発想が無く、意味の無い引数となってしまいました。


class内にはnormal会員とpremium会員の状態が混在しているのでMembersInformation,
functionが返すのはどちらか一方の会員の状態なので、MemberInformationとしました。


updating Sheet ID to new from corrupt/old


comment


나예찬 추가


That's a cool idea.  There's also this. http://stackoverflow.com/a/7735122
I think the idea is you put a tree listener on your view.  And it calls
your handler when the layout changes.

On Fri, Jul 17, 2015 at 11:55 PM supremearyal <notifications@github.com>
wrote:

> Yeah, the way it's structured right now touches invalidate the draw which
> generate draws later and we're trying to get two draws within a single
> touch event which doesn't work. This helpful post:
> http://stackoverflow.com/questions/4938822/how-can-i-use-the-animation-framework-inside-the-canvas
> has a trick where invalidate gets called on every onDraw which basically
> gives you a game loop and you can basically count frames and divide by 60
> to figure out when to draw/not draw giving you delay. But then again I was
> trying to avoid making onDraw a game loop.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/supremearyal/TicTacToe/commit/4de32143c78ead536bb0cb7fee38976394512b0a#commitcomment-12236686>
> .
>


Maybe replace by all by
Usage: !convert <float(aka number)>ft to yds
To prevent flooding which will get the bot kicked if multiple !convert help are issued


adsa


@ccx-testerbot runtests master


Thank you for the request. It has been added to the queue (id: 31). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/view.php?id=31) page. Please not that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


See issue #1 and #2.


I’m not sure I have an Off By One error here.

The code subtracts 1 from the array length, not the array index.


라틴어 부분에서 니켈너겟 빠졌네요. 수정해 주세요.


Лучше убрать if-else  и сделать #ifdef


asdasd


Neat, although it might be deprecated? http://stackoverflow.com/questions/7733813/how-can-you-tell-when-a-layout-has-been-drawn/27068733#27068733 I was thinking also maybe using a CountDownTimer to trigger a redraw which gives you delay. Basically queue up the draws yourself and use the timer to invalidate whenever you need to.


fkldlasdlkm,.as .da


would this run? It looks to me like mumble is unused


할만함
모르겠음


하나


둘


셋


넷


다섯


일곱


what if this lobby doesn't exist? It probably shouldn't happen but still, you should check for errors


This file was used tab indentation. But this commit use space indentation.


＊＊＊＊万円、＊＊＊＊千円という書き方はしないだろうという妄信をしました。単位を全てに設定すべきでした。


In the end it'd be funny if the compiler was "smart enough"™ to convert the generic c code that'd form this operation into the SSE code.


残骸です。消すべきでした。


I ran the code, and it works as I intended it to.

If there is only one entry, and the user inputs an invalid entry number, the message displayed is “Valid entries are integers 0 through 0” I could enhance the logic so that it would display “The only valid entry is 0.”


勉強になりやす！


Yeah, that's another problem with android is half the answers out there are
for deprecated classes.  CountDownTimer sounds good.  You might get into
some race conditions though.  If you change the state, set a timer to
invalidate, wait for it to draw, change the state again while it's
redrawing.  I don't know if that's possible.  I know JS is single threaded
so you don't have to worry about race conditions like that.  Any idea about
android?

On Sat, Jul 18, 2015 at 12:06 AM supremearyal <notifications@github.com>
wrote:

> Neat, although it might be deprecated?
> http://stackoverflow.com/questions/7733813/how-can-you-tell-when-a-layout-has-been-drawn/27068733#27068733
> I was thinking also maybe using a CountDownTimer to trigger a redraw which
> gives you delay. Basically queue up the draws yourself and use the timer to
> invalidate whenever you need to.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/supremearyal/TicTacToe/commit/4de32143c78ead536bb0cb7fee38976394512b0a#commitcomment-12236724>
> .
>


@bloodbare sorry about that. Though, I still don't fully understand what happened. PlonePAS was not in auto-checkout in buildout.coredev master and I removed it from the plip job cfg, right? This led to those multiple failures on the jenkins plip job that I thought were caused by Eric's commits. I guess what I wrote in my email to @ebrehault (reverting all commits to see which one was responsible), would have been the right thing to do.


あまり命名規則については意識していませんでした。


I quite like the indent but I'll stick to the convention.

I've updated all the files in 804d02582c41eca96364a7e3b434d1d1c30dd529.


Yep, might have to use locks. I know the view is drawn on the main UI thread, I guess the countdown is in another thread but I suppose it could be on the main UI event loop. Honestly going with a 60fps game loop and just doing updates in draw might be the easiest way to do things. If you have to do a lot of work like the AI, maybe thread that out but trying to keep draw clean might not be worth the effort for game stuff.


I did not realised that `ActiveRecord::ActiveRecordError` was the parent class of all the others. I've updated the code in 799d3ee694cd19ee7a01c9f9e41f120debb46435.


見づらくなるので変数に取りました。結果的に変数が多く見づらいので、変数に取ったところで見やすさは変わらないと今は思います。


よくわかっていませんでした。


特に意図は無いです。


I have found a regression that after bisection points to this commit.
The symptom that I am observing in my code is this:
TCP server sends a packet **larger** than 8192 bytes.
ZMQ_STREAM client receives a message of 8192 bytes. The message will be all zeroes.


Hey @xiam I realized the prior code didn't quite work once the release was tagged (for example in release-qa), i.e. GIT_REVISION_DATE would try to part a GIT_REVISION of something like 2.0.0-beta8, which wouldn't parse correctly.

I committed this directly in order to get the release out, but let me know if you think it looks OK!


try bit manipulation


Very smart solution


>planDetail.phpはなぜキャメルケースなのですか？
間違いです。


The lastest new about IESparser is it only can read the IES file, even i dont know how to use those source file >_<


And by the way, I think you should create a function `GetLobbyById` in lobbies.go. Direct mongodb queries shouldn't appear in controllers


OSX - clang does not yet support OpenMP therefore to build snesfilter you must use GNU G++ .. This compiler must be manually specified in snesfilter/nall/Makefile and I set mine currently to g++-4.9 which is for Homebrew


The test suite completed it's run. This is a summary (full info can be found on the [status](http://ccextractor.canihavesome.coffee/view.php?id=31) page:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **83/87** |
It seems there were some errors. Please check the [status](http://ccextractor.canihavesome.coffee/view.php?id=31) and [results](http://ccextractor.canihavesome.coffee/reports/31) page, and verify these.


ok, so loading message and loading file may different use case
1) loading message
when we want to change message contents, we'll change msg file and run
cmake to generate .l file. So we can assume .l file never changed. Changing
.l means changing .msg and cmake ran. So we can assume we restart roseus
program
2) loading file
I think in this case, we want to modifle file and reload them, so in this
case if we have old-module, then we remove, recompile and load them.

--
◉ Kei Okada

On Fri, Jul 17, 2015 at 9:00 PM, Yohei Kakiuchi <notifications@github.com>
wrote:

> If compiled messages are loaded twice, some error may occur. I forgot
> reason for that.
> In message file, depended messages is loaded, so the same message is
> loaded many times.
>
> By the way, old-module is checked before loading the module, so message
> should be compile for the first time loading message in the program. After
> loading compiled message, the same message is not loaded.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jsk-ros-pkg/jsk_roseus/commit/e7590d45f07039fa3b06f1de7b2a5f730a675444#commitcomment-12224076>
> .
>


an often overlooked trick


@jorgenschaefer this should be `'((cl-lib "0.5"))`


this should be fi


showcases algorithms for scheduling....


don't check for nvim this way because if it doesn't exist then this script will exit... also it should be `fi` not `endif`


Yeah, and if we used anvil we wouldn't have to redraw everytime.

On Sat, Jul 18, 2015 at 12:16 AM supremearyal <notifications@github.com>
wrote:

> Yep, might have to use locks. I know the view is drawn on the main UI
> thread, I guess the countdown is in another thread but I suppose it could
> be on the main UI event loop. Honestly going with a 60fps game loop and
> just doing updates in draw might be the easiest way to do things. If you
> have to do a lot of work like the AI, maybe thread that out but trying to
> keep draw clean might not be worth the effort for game stuff.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/supremearyal/TicTacToe/commit/4de32143c78ead536bb0cb7fee38976394512b0a#commitcomment-12236771>
> .
>


Looks like I've made a mistake here...


This package doesn't seem to exist. Has this change been tested?


Here you can do `GLFWInput(..)`


Isn't this just `GLFWInputT Identity a`?


Do we need this instance if we change the definition above?


Correction: Refactoring RAD-493


Changed formatting to be more consistent with other parts of installation documentation.


Need an instance for `MonadState GLFWInputState` ?


Here add some comments of what this file is for


woops added this fix @xiam


这只是一个最基础的版本，实现了从服务端获取数据，并将其解析。


should detect HOME automatically.


I think there is a problème with ignored words on libertalia. I set required french ans ignored vostfr for Marvel agent of shield season 2.and it downloads me vostfr 2x21 & 2x22


should be able to create these folders automatically, if there is none.


Do you see any issues with the queue being unbounded?


Pcbnew の ja.po の 2704行目と2726行目が未確定になっています。
どちらもUnitsとClassの表の所です。


This test is wrong. It's not `running_writer?(c)`, it's `c != 0`, since either a running writer OR reader will block us from acquiring the lock.


This should be `NewAddCOlumnMigration(monitorV4, &Colum{......`

L233 should also be monitorV4, Looks like it pure luck that it works using v3.


Imo, it's good enough to say 'web sites'. When we include `applications`, there's a risk that we encourage more use of dynamic component creation and requirements for supporting 'app-like' library features like observers, data-binding and so on. Imo, that's the type of thing best left to Polymer.


Thanks for adding this!


should the last part be 'promise of being lite'?


This is not re-entrant. If you nest critical regions which acquire the write lock, then yes, the *acquire* operation will work fine. But the *innermost* release will let go of the lock, rather than the *outermost*, leaving the latter part of the outer critical regions unprotected.

To really be "re-entrant", you have to count how many times the lock was acquired by the same thread, and only release it when the count reaches zero.


Using convolution with meanFilter:}

setRight(img.gray().meanFilter(4,'mirror')); //to mirror method
setRight(img.gray().meanFilter(4,'black')); //to black method
setRight(img.gray().meanFilter(4,'copy')); //to copy method
default: copy method


Where is the actual attachment getting passed in?


I need help please
I try to do it but I do how pleas help me!pleas!!!


Nevermind, just saw your next commit.


@ccx-testerbot runtests master


note to self - allow cancel and restart anywhere, not just in first place (but not both at the same time)


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


I wanted to php to change de mugi fuu pictures on site refresh. And add more waifus.


The "old" sorting assumed that all events of one block would come in sorted, as they were written out sequentially. This is not the case here because the event is written post-hoc (it cannot be written at startup without having any buffer allocated, nor flags parsed which tell whether or not it should be written).

IMHO it would be fine to swap those two lines in the reference if the new parser is going to handle this differently. This is simply what the new version does.
I tested EdenTV with the modified ordering, it does not affect its processing (maybe I will also change GHC to actually emit the events in meaningful order, but no need right now).

(ccing Thomas @horstmeyer for information)


Im using UpdateCSV, it works well in convert ies to xml scv. But cant makes ies file


테스트용


ㅁㅁㅁ


That's because your fish shell is old, update it.
`nvm-fish` is no longer supported, so nothing will be fixed anyway.


跟 rick 遇到同樣的問題，因為 isomorphic 的特性載入 css 需要在

`if (process.env.BROWSER) `

判斷式之內


Honestly I don't think it matters either way guys.  Plenty of software forces you to read news or messages of the day, it's not outside the norm and not a big deal.

On the flip side, i'm one that likes to read patch notes, change logs ...etc but I like the option of having to specifically go to the news page.  I think having to see it every single time would be kinda tiresome.

Perhaps a in-the-middle approach... since it should hit cdn.rawgit.com each time it starts kinda go back to what I mentioned over on the provider thread.  and have it "Watch for changes"  so on the next startup of the browser or page load after it detects a news change have it auto display that time.


Thank you for the request. It has been added to the queue (id: 33). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/view.php?id=33) page. Please not that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


general convention is to just have an underscore for mem vars.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.377](http://teamcity.monogame.net/viewLog.html?buildId=8899&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:07:12


[debugger] add Autobreak support

    Autobreak is a feature I have named for when you want to look at the data for a breakpoint which needs to occur potentially thousands of times, or more than you would want to do manually.

    I had experimented with updating the GUI but it was happening so slow that I finalized on a terminal-based approach. This means you must start bsnes from a terminal. Only tested on OSX, it can be done like this: bsnes.app/Contents/MacOS/bsnes

    Once you set your breakpoint, and activate the autobreak checkbox, the emulator will run in realtime as your breakpoint data gets logged in the terminal.. FYI, use Cmd-K to clear the console window before logging data.


you are initializing head twice here, did you mean to say ?
```
head = std::make_unique<DLNode<T>>(value);
tail = std::make_unique<DLNode<T>>();
```


this was not for adding the third curly brace. I just deleted the example above the actual template.


A cleaner implementation would probably have a private member func that tells if the tail has been initialized:

```c++
template <typename T>
bool _tailValid() { return _size > 1; }
```


this code looks correct, but the idiomatic way in c++ would be to implement the iterators, because you get contains for free with find();

```c++
template <typename T>
bool contains(const T& val) { return find(begin(this), end(this), val) != end(this); }

```


So it stops failing with different data?


damn, you're probably right, didn't think about that. We'd have to work with the typeof operator here. I'll give it a look later when I'm in front of a computer.


> 0 changed files
> 0 additions and 0 deletions

Well, hmm.


Welcome. But it is much nice,  that you work and help on the project.


@LinusU, I was wondering.... any opinions if we rewrite `_automaticErrorHandler` so that it can be added as method to uploadError?  Wouldn't it simplify deleting files when in `manual` mode? Example:

```javascript
// this function would need to be tweaked?
function automaticErrorHandler (uploadError) {
      function remove (file, cb) {
        storage._removeFile(req, file, cb)
      }

      removeUploadedFiles(uploadedFiles, remove, function (err, storageErrors) {
        if (err) return done(err)

        uploadError.storageErrors = storageErrors
        done(uploadError)
      })
}

function manualErrorHandler (uploadError) {
      // a callback that down-stream middleware can call to make deletions simpler
      uploadError.automaticErrorHandler = automaticErrorHandler;
      req.uploadError = uploadError
      done(null)
}
```


pinning using the shell works but file only ends up in 'All' tab from the webui. pinning before implementing shell would put file in 'Pinned' tab as well.


val seems unused.


We'll see


I think you're missing the closing quote for the 2nd `grep`.


我倾向View 作为调用方不用知道 container 的存在，直接使用 opener.on('action@actionevent'); opener 内部自己做好 action 事件和自身事件的分发；


Oh, really? That's too bad. I found it very helpful. Do you know what is it replaced with, if anything?


Ouch... Yes, fixed in #47


revert package change


great


In the US we tend to spell it liters and not litres. For whatever odd reason.


> viewContext的逻辑似乎应该在DrawerPageOpener里面才对，但是相互怎么关联会成问题

 因为使用了组合的方式做了子页面的事，导致控件和 view 之间隔离了一层；
 其实，我觉得openPage 这个方法直接返回一个 ActionPanel 的子类是不是应该就够了，getInnerPageOpener由业务方根据需求去重写，ssp 这里直接返回 this.create('DrawerActionPanel'), dsp 可能就是this.create('DspDrawerActionPanel')


Wow, that was a lot of misspellings. Thanks for the corrections.


Revert this


All caps is like yelling. Search engines considers it as abuse. So, have as normal text. Make it all caps using CSS


Please check on mobile screen.


I'm just trying out the comment capabilities. So don't pay attention.


Well, no. Don't you want to alert the user if they gave the wrong numer of argumemts to an instruction? Besides, giving less srguments than bot.cpp expects can lead to a segfault. This is my way of preventing that.


And here is a line note?!


Why is the CellML solver disabled here?


Also: `WriterLock` could be better named `WriterThread`, or `WriteLockHolder`, or something like that. The current name is hard to understand.

One more thing: Would it not be better to make the *reader* side re-entrant as well?


The closing parentheses are important, since it closes the line for the subshell `$()`, for eval.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **83/87** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/33) page, and verify those files. If you have a question about this report, please contact@wforums.


@charleshansen hey i was looking at adding bluebird support to mock-promises, it looks pretty straightforward, these are the only code changes I really needed to make, I had some questions about how your setup the test but before I go through all of that did you have any better ideas on handling the way bluebird needs to get the fulfilled or rejected values?


Correct, sorry - long day... Hopefully now it looks ok.


@ccx-testerbot runtests master


Thank you for the request. It has been added to the queue (id: 34). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/view.php?id=34) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


so fast !


Not unsurprisingly, there is a fair amount of duplication between the Newton and Quasi-Newton methods/types. This probably means that the common variables/methods should be put into more of a "base" class/type.


時間が確保できないとかそろそろ分量が多くてきついとか
きっと色々あって大変なのでしょうが、もう一歩
丁寧に書ける様になると良いですね。
名前や消し忘れ、フォーマッタ等等。


フォーマッタに関しては不備の回数が多すぎるので
自動化をした方が良いかも。[参考](http://qiita.com/sisidovski/items/bde2d844c3c73457923c)


It could use a lot of memory if the main thread doesn't consume fast enough. Due to Neovim architecture(multiple event sources, sometimes one is "focused"), I can't place a limit on the queue because blocking the IO thread could result in a deadlock.

What I can and will do is put a limit on how many events are placed on each event emitter's individual queue without it stopping the read. So for example, if nvim is blocking on a single channel, the other channels won't generate too many events.


`I like the option of having to specifically go to the news page. I think having to see it every single time would be kinda tiresome.` +1

Its not using rawgit anymore, because www.rawgit.com is limited (for non-production) and cdn.rawgit.com only provides 1 static page per url, cached forever (unless you pay $9 a month). rawgit just isn't an option.

On the other hand, we can add a repo here with changes.md and news.md served from gh-pages branch
@fernandog needs to create the repo, git me access, and then set gh-pages as the default branch


Does it work?


I would recommend to create a Metadata_View that inherit from View.
And you have just to create the Metadata_View in the SfMInit_ImageListing.cpp
In this way we willnot break the existing API, and have the basis View class that just embed the necessary, and show how to manage custom user views.


I'm not sure why we need element parameters in local line/face??? Why can't these be obtained from the node and derivatives???


This will go so wrong. When you leave scope on L563, `left` will go out of scope and `l` becomes a dangling pointer, probably causing a segfault later on. Nice attempt to reduce copying, but this doesn't work.


Can you change the link also so it points to your username and not Darren Oakley's? Also there's a misspell. Should be: `Contributed by [f440](https://github.com/f440)`


no, I'd try it. It not look like nice with the popular ext ThanksForPost
Now we haven't event for this. I think to fix the place with javascript


Please change the name to your own here too.


Besides, here you suddenly assume `l` and `r` can't be `NULL`. :')


could fail if head isn't initialized(i.e. the default ctor was used)

```c++
#include "DLList.hh"

int main() {
  auto dll = DLList<int>();
  auto v = dll.back();
  v->value();
}
```

```bash
[list]: clang++ -std=c++1y test.cc -Wall -Wunused
[list]: ./a.out
Segmentation fault (core dumped)
```


This is assuming a quad element here?


K, I'll delete :P


```c++
if (!head && !tail)
```


sorry about that, this looks like a merge error. PR coming to undo these lines!


サインイン後にしかアクセスできないリソースは全て `my` namespace 以下に置いています。
そもそも、他のユーザーの情報はサインインしなくてもアクセスできていいじゃんという考え方もありかとは思いますが。
namespace を `my`  以外の何かに変えれば、違和感なく読めるかもしれません。
すぐには思い浮かびませんが...


This commit message is cringy. "Add rank list command" would be much better.


No -- this shouldn't build when you download from hackage.


this looks right, but could probably be implemented more cleanly with std::swap


Nope

Varun Jain
LinkedInProfile <https://www.linkedin.com/profile/view?id=69628206>
#: +91-8376846196

On Sat, Jul 18, 2015 at 11:21 AM, Utkarsh Bansal <notifications@github.com>
wrote:

> Does it work?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/trolltartar/Newsybackend/commit/151563ac0f7972c742b5b128674d3cc130e7791f#commitcomment-12237011>
> .
>


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **83/87** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/34) page, and verify those files. If you have a question about this report, please contact @wforums.


use p ...


Same


It won't work because after the first cd, the others are staying in the app/assets directory.
`( cd app/assets ; npm install )` <= launch a sub bash so won't change the current directory


ddd


Don't use HTML whenever possible -- jade has support for the `li` tag ;)


Sane here -- this is inconsistent.


把import alloy-editor-all-min.js的方式改成用require,放到process.env.BROWSER的判斷式內後，出現以下錯誤碼(註1)，看起來是babel在compact的時候超過100KB，導致alloy-editor-all-min.js檔案被截斷，所以出現如圖中的紅色字error ....。

有找了一些網路上的資源說把babel 加上 compact: 'false'  等等的方法,不過我這邊測試都還是沒辦法。
http://stackoverflow.com/questions/29576341/what-does-the-code-generator-has-deoptimised-the-styling-of-some-file-as-it-e

---
註1：
[BABEL] Note: The code generator has deoptimised the styling of "/home/chrick/posteditor2/mobious/node_modules/eslint-loader/index.js!/home/chrick/posteditor2/mobious/app/postManager/alloy-editor/alloy-editor-all-min.js" as it exceeds the max of "100KB".
---
![default](https://cloud.githubusercontent.com/assets/3950196/8760539/49f14b6a-2d55-11e5-8a98-0287129ac060.png)


@ichekrygin It is written in description, there are [nvm-fish-wrapper](https://github.com/passcod/nvm-fish-wrapper) and [bass](https://github.com/edc/bass).


Fixed at `47b9d2365b`.


Geoff, I think he has a decent point, and I think that the fact that we've changed the parsing, to use the "NoStringtableLookup" keyword (or not) allows us to explain fairly clearly how any why we used a stringtable key in that sitrep but not in the other sample sitreps.  Ouaz, I also made the 4 preset labels translatable as well.
I've added a fair bit more explanation in the custom_sitreps.txt file; Ouaz I suppose you could consider adding some of that material to your translation of the pedia article, so that it can be translated there.

As for removing or commenting out the other examples, I think they are helpful enough that I would suggest actually leaving them in, but my second choice would definitely be commenting out rather than removing.


Nice job here with the method syntax...will take some getting used to coming from mainly C# constructs at LifeYield.

In C#, the method would appear as follows...

```
public int getLengthOfStringForBool(string theString, bool theBool) {
    if (theBool)
        return theString.Length;
    else
        return 0;
}
```

Have a great day!


Está bien! Los inputs mas pequeños nomás :)


pre-git is what handles all the git hooks
On 18 Jul 2015 00:41, "Oliver Faircliff" <notifications@github.com> wrote:

> I needed it to commit
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itsapi/github-listener/commit/095f2addf9b8599227c1370a6eb3b9beae8b7bf7#commitcomment-12235525>
> .
>


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


有2个问题：

1. 如果用控件子类，我们在哪里做`require('XxxPanel')`的工作？
2. 除了抽屉外，最常见的必须覆盖的交互方式是Dialog和直接跳转，使用子类的方式似乎并不能很好地覆盖直接跳转的场景


Thanks @xiangzhai going to do a review this morning


Solução: http://www.eclipse.org/eclipselink/api/2.1/org/eclipse/persistence/annotations/Mutable.html


会有些问题，这样的话我们注册事件的时机会不一样，是不是得在`setXxxOpener`的时候就注册好这些事件了？那么在`setXxxOpener`的时候，由于View并没有实际渲染，是不是有足够的上下文让我们搞定事件处理函数呢


OK


look how it realized in 3dmoo:
https://github.com/plutooo/3dmoo/blob/master/src/arm11/thumbemu.c#L200
(both 3 cases realized).
AND another thing about thumb!
BX PC here
https://github.com/citra-emu/citra/blob/master/src/core/arm/dyncom/arm_dyncom_interpreter.cpp#L4148
must add 4  to PC,
this is fallthrough from THUMB to ARM mode. I think, instread LOG_WARNING should be something like
cpu->Reg[inst_cream->Rm] += 4;


=w=


This is a simple coding task that I did during the slacking days of summer.
I wrote the code in Python and added simple GUI using Tkinter, to do away with the CLI (It increases the audience considerably.)
Some suggestions that are to be implemented are as follows:
1. Activation of the enter key to submit the guesses.
2. Adding the graphics of a hanging man.


This is hard coding. Please fix it. Also this code should not be aadhar xml parser


@Sanjo tested in Simian?


Let's open an issue about it to ensure we don't forget to deal with this.


These we unneeded, so removed.


人家就是喜欢你讲

2015-07-15 1:19 GMT+08:00 Teddy <notifications@github.com>:

> 自己试试就知道咯
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/teddy1004/wx_pay/commit/6a33aa180ed92bd674982fc1a66951ae80412349#commitcomment-12160863>
> .
>

--

yours,

正中赵
about.me/zzz6519003
[image: 正中赵 on about.me]
  <http://about.me/zzz6519003>


@paul-lrr While there probably isn't a good reason to switch away from lrrbot for sub notifications at this point, I thought it couldn't hurt pointing this out to you. With the new WebSocket support it's reasonably easy to grab the information directly from Twitch chat. I prepared a [rather crude example](http://pyrige.github.io/cstsn/lrr.html) that works perfectly well as an XSplit source (at least for me).


>lzip
3am is 2hard
zlib*


> 如果用控件子类，我们在哪里做require('XxxPanel')的工作？

业务端自己去 require， 基类提供一个接口： getInnerPagePanel, 业务端重写这个，返回一个 ActionPanel 子类；

直接跳转的场景，业务端重写 openPage ： 丢给 action 做 redirect；


Code changed for My CNC settins.


@mailwl Nice catch! Thanks


Test submitted:

http://tests.stockfishchess.org/tests/view/55a9f5a30ebc590abbe1bb2b


这些我觉得opener 自己内部处理好时机？策略上根据 action@ 前缀来区分， 对于 action 事件opener 仅仅是需要将 actionPanel的事件对外代理出去


@mailwl To clarify, are you talking about increasing the value read if the source is r15? (As in `bx pc`)


I like this


我们就是要业务端不重写的情况下来实现，才会使用组合的方式

在模块化的前提下，业务端不能有自己的Base基类，所以不适合做重写的工作

那么，直接跳转时：

1. 怎么丢给Action，事件会导致异步而无法立即返回所谓的Panel
2. 这个Panel又是怎么实现的，所有方法置空？


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


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


Would you mind updating the pull request without these 3 blank lines? Otherwise looks good, +1


一个opener是可以产生多个Container的。我还是不明白我们在什么时候去opener上注册事件，因为opener是一早就存在的，那么我们是在一早就注册完事件吗？


yes - PC on arm processor shows not current instruction offset, but 2 opcodes later (historical reasons with instuction cache). so, for THUMB mode BX PC real mean BX [PC]+2+2, in ARM mode this is not used AFFAIK, but not sure (in this case PC+4+4)


which a browser


Please add Europe/Chisinau


github commit


Also, reminder to self: don't try to amend a previous commit after you've already pushed it to github, because git does not like it one bit.


i saw this in game Conception II, IDA Pro shows where lead BX PC (next after this instruction usually THUMB nop). But Citra just jumps back to itself with changing mode, but not invalidates instruction cache, and then just infinity ARM/THUMB jump to itself (if i not mistake)


Correct! Also, you have `Integer.MAX_VALUE` for the ma integer value


> 我们就是要业务端不重写的情况下来实现，才会使用组合的方式
> 在模块化的前提下，业务端不能有自己的Base基类，所以不适合做重写的工作

如果是模块化有弹出子窗口的需求，模块的 mvc 去重写应该也没啥问题吧，并不一定要业务 Base 去重写？

> 直接跳转时：

我是这么想的，openPage/forwardTo 就不弹出窗口了，直接 fire 一个redirect事件给 action，action 监听这个事件然后跳转；
如果为了一致性考虑，可以实现一个 RedirectPanel，只是没有视图而已；


Exceptions! :+1:


What exactly doesn't work?


hi


`a = null;` is not necessary which makes this function rather redundant


bro see its right?


我们的问题是，同样一个模块提供了列表+表单页，在A系统可能是抽屉出表单，在B系统可能是直接跳转。列表如何跳转到表单由系统决定，而非模块，但模块是系统共用

无论如何，用抽屉还是对话框还是直接跳转肯定是由系统来决定的，但因为模块化的架构，导致系统是不能有自己的Base基类的（不然会导致独立模块提供的MVC类的继承问题，以前有说过），所以系统中并没有一个合理的地方去重写基类提供的方法，我们只能用`SubmitHandler`一样使用组合的形式

回想一下我们为啥会有`SubmitHandler`而不是让系统自己去重写`redirectAfterSubmit`就明白了


<Class ClassID="11410" ClassName="AggressiveShot" Name="<$>63511</>" EngName="Aggressive shot" FileName="stn_standingshot"
You can see Targan shot is using standing shot icon. Correct icon is stn_assaultshot
<Class ClassID="11420" ClassName="InroadShot" Name="<$>63513</>" EngName="Inroad shot" FileName="stn_kneelingshot"
Meanwhile Straight Shot is using KneelingShot icon. Correct icon is stn_snipingshot


Your approach is fine, correct but might be slightly inefficient. You only care about the numbers which are on the diagonals of your matrix. Is there no way to compute just those numbers without having to fill up the whole matrix? think about that


hier musst du noch die False durch True ersetzen, da sich ja die Logik vertauscht hat


Check out [this post](math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) to see how to compute this in constant time instead of `O(n)`


Handled inside of the shim now


reack


Careful on the complexity here. Check out [this post](math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) to see how to compute this in constant time instead of `O(n)`


Great work, nice formatting and code organization.


試してないですけど、`self.search_linear()`の第５引数にendをそのまま渡すのはどうでしょう？


> 一个opener是可以产生多个Container的

这个我倒是没有从代码看出来，因为 opener.getContainer() 看起来就是只有1个 container？

> 我还是不明白我们在什么时候去opener上注册事件

这个应该没有太大关系？ 我理解 opener 内部的策略是这样的：调用 opener方法注册事件时， opener 会根据 container 是否渲染了来做处理，没渲染则放入队列，渲染时将队列的监听信息给 container 丢上去；


Students/Farcas Silviu Vlad/Assignment 2.1/Polynomial.java

I wouldn't really construct it like this. Think about a polynomial as having at least 2 properties:
- degree
- a list of coefficients.
I would expect a class which models such an object to have at least the possibility of giving me its degree and maybe even a list of coefficients (not necessary). Right now, I can't really do anything with your polynomial class besides printing it nicely. Yes, this is a great method to have and it should stay here, however, you should encapsulate more relevant data in it! Coefficients, degree etc.


Sry messed up uploading file


@twizmwazin What would you suggest? I think the commit message is good enough.


Yep for pip installation. That script uses root


Students/Farcas Silviu Vlad/Assignment 2.1/Functions.java

`add` function should take as parameters 2 polynomials and return a third


`subtract` function should take as parameters 2 polynomials and return a third


`mulScal` function should take as parameters 1 polynomial and return a second


`multiply` function should take as parameters 2 polynomials and return a third


`eval` should take a as a second parameter a polynomial


ah - alright


Also, this should take as a parameter a polynomial


这个代码确实有这方面的问题，我理解应该是`opener.forwardTo`会返回Container，而不是再用个啥方法去拿……这个opener对应的是我们现在的`openDrawerActionPanel`，当`id`一样的时候会复用，不一样的时候就可能打开多个了

基于一个opener会有多个Container这个前提，如果事件在opener上，会变成一个实例代理多个实例的事件（虽然可能这多个实例不会在同时出现），那么代理出来的事件可能会无法区分来自哪个Container


reack


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


cool :)


it's correct!


Your README file should be a description :) kinda of what you have in the header.


assert 処理を改良することにします


Rule of thumb. If you're modelling a method which should do something with two polynomials, it should take as parameters these two polynomials. Yes, those two might be perfectly described by just a list of coefficients, however, you can simply encapsulate that list inside the `Polynomial` class and even give it another `degree` field so you don't have to use the `coeff.length` each time. It's not really elegant at all.


Students/Farcas Silviu Vlad/Assignment 2.2/MatrixOperations.java

Yes, correct, but if you're having a `singleton` object then why are these methods all static? That means you don't have to instantiate an object of this type and you can simply use `MatrixOperations.add` for example (note, I didn't create an object of this kind)


Well done, but improvements are needed


but needs a check if mongo saves dates in utc we need to use DateTime.NowUtc


I like this name :)


Could MPDN directly activate the debug dialog when MPDN is launched with the /DEBUG from VS ?
Because when MPDN got a NPE, I didn't get the full debug dialog when launched from VS, just the Error box.


저 부분은
>> 이렇게 바꿔보는건 어떨까요?


아 그렇네요... 감사합니다.


done


why it's simple?
It's a filter by name, right? why don't name it GetOfferNameFilter?
since minSalary, maxSalary aren't used it would be good to remove it at all


you're using same string in many places,
consider about enum, or const string,


test


when you change schema, it hurts,
you can use attribute [BsonElement("IdRecruiter")]

but still it's good to have same names in db and in code


Ideation Portal Java implementation complete code available here.


Looks good :+1:


Updating comment as per https://github.com/bmildren/percona_mysql/issues/4


Maybe you can also update the Radxa table on the website including the special pins?


can you sum up the changes?
i don't get the change in object.class.
Just to output automatically the properties if they are present?


I think they land in couchlog and was hoping the log files - though not sure about that.


@sheelio can you explain why you need child cases of a user case?


Won't these needlessly parse tags against the linked player, in addition to parsing per-player?


please ignore a.html


why we don't do the same modification to this resizeFilter like for normal filter?


asd


It shouldn't. To even be able to parse a tag for the linked player (that isn't in the players: argument anyway) you'd need to not have per_player. Unless I'm missing something?


Every command preparses arguments unless you set it not to. It's part of the core script engine, it's not done in the command itself.


Right. How do I set it not to then?


```
    @Override
    public void onEnable() {
        setParseArgs(false);
    }
```


Your fork is an exact copy of mine but for some reason Github decided that it shouldn't load and I don't want to have any down time so it redirects to mine until we can make it work. Sorry!


> 不然会导致独立模块提供的MVC类的继承问题，

你说的是对的。

用控件方案的话，我是想业务端将控件类型字符串作为一个依赖注入进来，getInnerPagePanel里面是 this.create(this.innerPanelType);

 exports.openPage = function () {
       var panel = this.getInnerPagePanel();
      delegate(panel, this, 'redirect');
}
这样我觉得比包一层opener 要处理的琐事更少些。

如果采用 opener 的方案，就 viewContext 的问题，直接把 view 作为 pageOpener的依赖传入如何，opener 里面调 view.create去创建容器；redirect 的场景直接给外面抛事件，view 代理到 action 跳转？


Hello

Please contact me to "eranzilber@walla.com"
I found the buttons combination to mount all the partitions (boot, service, kernal, rootfs, korg system)

Thank you


div.sidebar table {
	margin: 0 auto;
}

use code above to center <li>


Board Object 1개를 가져오고 싶은거라면 objects.get() 함수를 사용하면 될 것 같습니다.
https://docs.djangoproject.com/en/1.8/ref/models/querysets/#django.db.models.query.QuerySet.get


오타 있네요


Ended up being a user permission in MySQL. I forgot to include permission for INSERT -_-


Yeah, i think.


Yeah.

Csabi

On 17 July 2015 at 20:01, Yoann Delouis <notifications@github.com> wrote:

> In the constructor and in addAllSupportedOptions ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yDelouis/androidannotations/commit/2dbfdd5a230fca44cab3c9cc6666d5a7d15d17cf#commitcomment-12230496>
> .
>


That crossed my mind when I started it and it really seems like a better approach. I think I just wanted to see the whole matrix. Change it, I will. Oh, and do I have to upload the whole project again or just this one file? Or is there a way of changing the code here on git without having to make another commit?


@yDelouis i think you missed this comment.


I'll take a look at that.


@cebe do you suggest reverting it?


okay


asd


Nice, but this method is getting a little long. It also has a high complexity. Look into **rubocop** and **codeclimate** — these are commonly used tools that give great feedback.


Эти функции определены в стандарте ECMAScript 1. Вроде везде должно работать.


I don't think you had the latest version when you copied the existing controller..


Beautiful. I like that the three phases — setup / do the thing / test — are clear to see.


This line should be removed.


i have no idea how this happened


Oops, fixed. Thank you! :-)


'vh' and its kin cause problems on iOS ~7 and some IE versions, would that be a problem?


Related note @ChrisKitching: Targetbones should be relatively stationary relative to the origin bone, during unit movement. Otherwise this causes a 'wobble' with beam weapons, as they try to aim at the moving bone throughout the target units movement animation. This will occasionally cause the beam to miss.


サンプルエフェクト（スクイーズ）を作成してみました。よろしかったらpullしてください。


Simply change that file, commit it and push it. The new commit will pop up here, on this pull request


よかったらpullお願いいたします。


Why changed it from SpreadSmile ?


Wp.


If UNLIMITED_SIGHTRANGE equals 4999 then it's correct from a logic standpoint. It looks weird however because what could be bigger than unlimited sightrange?

btw: I think sightrange is a bad translation from "Sichtweite" (German). It should be called "view distance".


hmm, I synced with your code that you checked in 30 min ago. here is the
copy of my code that i see locally

  <<Override>>
  public function execute(\Core\Context $context): void {
    // The following controller connects to Provider's dashboard model
class.
    // There is no relationship b/w model and view in this example.

    (new \model\DashboardModel())->getData($context->m_request->m_params);

However, what is checked in seems different. I have no clue how this is
even possible!

On Sat, Jul 18, 2015 at 1:16 AM, imback82 <notifications@github.com> wrote:

> I don't think you had the latest version when you copied the existing
> controller..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sooryav/hippo/commit/fe027f8f519e523fb94aaa7770cd36fcfa42f8ea#commitcomment-12237369>
> .
>


I believe it is due to dynamically added content which shouldn't interfere here. It should degrade to what it was before, at least in iOS and at the worst it will be longer than the 80% which takes us close to what existed before.


Removed 204-124ST


would upstream need to worry about this?


ADADS SAFSFSFE AFDFG


@TaylanUB, this commit broke the tests. Could you please check what needs updating here? Thanks!

https://travis-ci.org/jorgenschaefer/circe/builds/70634603


@banjaxbanjo  What does that mean? Is the function used somewhere or not?
Also it seem to be a funny name of it. Misspelling? (What's the "P" for?)


Because we use moq, it should be "public" here instead of "internal".


In console without the color code use the &7 code by default


Chat is already &f by default


Should it be max-height instead of height? A user may have few subscriptions (or very tall screens).


This is really cool, how you can do it in SQLIte. What you need is essentially a join and this is one way to do it (as long as column names are unique).
```sql
SELECT DISTINCT MCC,MNC,LAC,CID,gpsd_lon,gpsd_lat,rx_signal,time,gpsd_accu FROM DBi_measure, DBi_bts WHERE isSubmitted <> 1 ORDER BY time;

```


Indeed. I wouldn't have removed the error handling, but there were no tests for it. =)


Neither. In fact, the truth is that this place should handle encoding errors etc, so another thing to add.


Can you provide logs for this? It sounds more like a SickRage issue for me.


Change screen orientation, so that no need to handle configuration change event.


@E3V3A I did not create this eva and as it stands now its not working over not having right columns. Its used for getting data to upload to ocid.


@YannBertrand , I think the view is loaded before the scope. I will dig into this later


@E3V3A thats cool so this will return a cursor with these in this order yes?


Please edit your messages instead of posting new ones, or if it is a different issue open another one.
I didn't understand what's your problem, did you even try to compile it? This is a parser, so of course you can't go from XML to ies.


Calculate how long since we fall in love.


it was a test commit will be reverted back next time


cool!!


Make handler static, enable that click actionbar home button to show slide menu.


(took conversation to the spec)


I found that if the OPTIONS request pass,allstream(android)will auth failed.


@hvelarde It broke one of my project on travis, I don't remember which one. I don't think I use coveralls indeed.


What is this doing here? We should not get any MAX_VALUES or `-1` for CID...
We already have a function to remove bad data somewhere...I forgot where, perhaps in the ocid download?


added the `--utc` flag too


See my previous comment above


Why not just `if (true == plusEnabled)` ?


post를 받아오는 것 때문에 코드가 지저분해진다고 생각하면 forms.py를 쓰는것도 방법일 것 같습니다
https://docs.djangoproject.com/en/1.8/topics/forms/


Apparently not? I have no idea how rawQuery handles `;`s...


is this program more correct ?


ack


@E3V3A also did not create this eva only updated table names and columns,looked like a weird query so I left in.


why u do like this


\o/ great! if you don't get to it I'll take a look on the train this week.


这样会导致close阻塞，是异步化diskwrite变得没有效果。
后续考虑提供两种close语义，一种阻塞的，一种非阻塞的，当前只实现了非阻塞的。
我使用了leveldb的lrucache，相对于自己实现的，会更完善些。
另外将读和写分别打两个文件描述符，实现了互不干扰。
https://github.com/bluebore/dfs/commit/0531b6a25efd290f99df86f7630fef5477941180
小数据量测试没问题，大数据量测试还在进行中


I can't  find the style.css for your examples. It is shown as contained in /resources/demos.


reack


Nice idea to create the mountpoint if it doesn't already exist, but please add a message which informs the user that the mount point was created.


No. I rebased my branch so it has been changed in a new commit. That's why this github page did not change.


Credits:

Due credits are to be given to the creators of py2exe. It was because of them that THIS can be run even on systems that don't have Python directories installed in them.


Same comment than above.


@jpetrie - oops, that one slipped away. There *was* code in there - I'm going to remove it. Thanks for spotting :+1:


Great, thanks!


Please use `cmd.global.Verbosef()` instead of `fmt.Printf()`, e.g. like https://github.com/restic/restic/blob/master/cmd/restic/cmd_backup.go#L268


@AbdealiJK those are two commits, you're doing the mkdocs stuff also here.


We should create a `done` channel and use it here instead of `make(chan struct{})`, which creates a channel that is never closed. The usual pattern can be seen here: https://github.com/restic/restic/blob/master/cmd/restic/cmd_find.go#L182-L184


Same here, needs a `done` channel which is closed with `defer`.


Good point, maybe `partialField` should be exported. Will be a future consideration.


It's okay for now, but we might add a `map[name string]struct {snapshot, timestamp}` or something like that, which caches the snapshots for a while (like 120s or so).


可以通过异步执行LRU的callback来解决，我正在处理merge冲突，打算处理完之后再改进LRU。
既然这样的话，那关闭这个PR好了，刚好可以学习一下leveldb中的lru实现~


You are repeating
```
[KSPField(isPersistant = false)]
public bool plusEnabled = false;
```
and
```
if (StockPlusController.plusActive == false || plusEnabled == false)
 {
    plusEnabled = false;
```
over and over again. If I were you I'd make a superclass. Doesn't KSP allow this?


What's this used for?


I don't understand what this is used for. Is this the way to tell the fuse library to use the dir struct for `ReadDir`?


Hi totakke, the link is not available, could you help me with this? thanks.
eri-129-70:~ cai$ brew install nite
==> Installing nite from totakke/homebrew-openni
==> Downloading https://onedrive.live.com/download?resid=33B0FE678911B037%21573
==> Downloading from https://onedrive.live.com/pagenotfounderror

curl: (22) The requested URL returned error: 404 Not Found
Error: Failed to download resource "nite"
Download failed: https://onedrive.live.com/pagenotfounderror


You could just return `node.Size` here...


we'll probably want to use None here again


this comment should probably be removed


Thx


This log printer actually drops the order of the log messages. Why don't you simply create a list of LogMessage objects instead of doine the whole dict thing while losing all kinds of information?


personally I like mor flat design. I imagine Lcars to be made of a flat enlightened screen, more than actual 3d objects like this one, I'll try to make a version of this to show what i mean, but if you want to switch eruption and start to include 3d stuff, no problem, but I'd definitely stick with a main guide for the look and feel of the skin as a whole...

let me know what you think!


you don't need the grep anymore


@scalar77 무슨 일 생긴 건 아니겠죠 ㄷㄷ


these lines usually appear when solving conflicts with git. Did you do anything special with git?


まずは音が出るかの確認用。
音色はランダムに選んでいる。


self-ack empty commit


Done.


イラストレーターで、コピペする時の対処法。


I was mistaken, alignToInterval only does this:

 if alignToInterval:
    requestContext = requestContext.copy()
    s = requestContext['startTime']
    if interval >= DAY:
      requestContext['startTime'] = datetime(s.year, s.month, s.day)
    elif interval >= HOUR:
      requestContext['startTime'] = datetime(s.year, s.month, s.day, s.hour)
    elif interval >= MINUTE:
      requestContext['startTime'] = datetime(s.year, s.month, s.day, s.hour, s.minute)


love you, mate.


Yup. I know.


Yes and no. :)
Please see: https://github.com/SecUpwN/Android-IMSI-Catcher-Detector/blob/f8d420423b1518351fdfdd57cb431a4b27088cc9/app/src/main/java/com/SecUpwN/AIMSICD/service/SignalStrengthTracker.java#L29-L60

Summary, yes, we should change all timestamps to use INTEGER **and** only use times as [seconds] in Unix Epoch format. This will allow us to use the SQL as tool and avoid shit loads of java.


And the effect for non-sbus RX that work fine with 985 ? Failsafe is no longer safe?


I implemented this fix for the [2.0 branch](https://github.com/Codeception/Codeception/commit/fa31fa18696bc96250e4ebed1517e0b2443844ce) and the [2.1 branch](https://github.com/Codeception/Codeception/commit/d7d2832900988a95046d10a03210de002d251cd5). Will be available in the next releases. Thanks!


This is my first time with git. It is very possible that I may have done something wrong, but I do not know what


desc : "This is an unidentified quest"


This was pretty stupid way of animating parts of loading control. :blush:


no


Delete file


Hmm, I'm not sure if I like exposing methods on the error object.

Is your use case that you want to be notified of the error, but still have the files removed for you? Maybe we should split the options into two then; `errorReporting` and `removeFilesOnError`?

Or we could expose `.removeFile(req, file, cb)` onto the multer class, which would just forward it to the storage.

```javascript
Multer.prototype.removeFile = function removeFile (req, file, cb) {
  this.storage._removeFile(req, file, cb)
}
```


Yeah, we can drop it. I've removed develop.


@Velho : the change in this commit is to make the entire source follow the same standard.
Would you rather like that the source code is a mix of CR/LF and LF-only text, randomly based on what each user chooses to set in their version of git?