labofsahil/github-event-dataset-2012 huggingface dataset:
Introduced a Server monad to wrap the server state

The code compiles and runs. See the updated Counter.hs file for a simple example. 

Server handlers are now implemented in terms of a Server monad, which wraps the Process monad. The state can be manipulated using the helper functions: getState, putState, modifyState.

E.g., the following implements the Counter server:

```haskell
handleCounter IncrementCounter = do
  modifyState (+1)
  return $ CallOk (CounterIncremented)

handleCounter GetCount = do
  count <- getState
  return $ CallOk (Count count)

handleReset ResetCount = do
  putState 0
  return $ CastOk
```

The inferred signatures for the counter services would be something like:
```haskell
handleCounter :: CounterRequest -> Server Int (CallResult CounterResponse)
handleReset :: CountReset -> Server Int CastResult

-- based on GenServer's handlers:
type CallHandler s a b        = a -> Server s (CallResult b)
type CastHandler s a          = a -> Server s CastResult
```

Still lots of small todos such as the proper termination, handling of timeouts, and handling of all InitResult, CallResult, CastResult.

No attention has been paid so far to proper file/code formatting, documentation or even more idiomatic Haskell code.
Nokre typos

Fiksa nokre typos. 
Ver konsekvent på bruken av "nestede"/"nøstede", burde kanskje berre bruke ein av dei.
Jamnt over ein interessant og lettlest bloggpost. Tommelen opp! :)
enhancements and cleanup

This depends on quicksilver/Quicksilver#1082 and should address the things discussed on quicksilver/Quicksilver#851.

  * take advantage of the new `QSSearchURLType`
  * make the Web Search plug-in recommended (as it always should have been)
  * project cleanup
Change red built-in constants to soft purple.

Change red built-in constants to soft purple.
Added G403Translator
LPS-25635
override handler filters which are not appropriate for metric handlers (librato handler)
LPS-25382
LPS-25226
Added test typing non latin text

Added the test for
https://github.com/tkurnosova/phpunit-selenium/commit/ac82e49c737d69e4e7a346c24ee217fe2749d2c9
moved colorscheme to colors folder

Hi,

I added a colors folder because it makes the color scheme easier to use with pathogen.

Merge it if you like

Anders
Removed Icon Preview

Removed Icon Preview by adding an _ to the 2 Preview files.
Fixed typo in PluginController

Fixed typo in JMS\Payment\CoreBundle\PluginController: $instructin to $instruction.
Peformance optimization to BoundarySet resource

I was running into some performance issues with boundary-set/ URLs -- they were doing the equivalent of SELECT * FROM boundaryservice_boundary, which meant asking PostGIS for a lot of rather hefty polygons.

This changes that to a SELECT id, slug FROM boundaryservice_boundary, which in my use case helped performance quite a bit.
diff

377行表意不清楚，不知道怎么改。
Development

Multiple bug fixes for logged JIRA issues.
remove t in MVCt on line 277 column 413

I belive the t in MVCt is a typo.
Ensure result before trying to use it as a Hash

Was getting `NoMethodError: undefined method `[]' for nil:NilClass` when there was no more jobs to process, this ensures that the result object is not nil. :dancer:

cheers
About page mods, blue demon weekly link, various styles, Added podcasts to sports page and fixed up schedule
Première version 

Salut,

Nicolas et moi avons terminé une première version fonctionnelle de l'application. Il est actuellement possible d'effectuer une recherche selon les critères sélectionnés. Nous n'avons pas eu le temps de tester dans tous les navigateurs, alors il serait bien de vérifier que tout fonctionne correctement.

Choses restantes à ajouter/modifier:

- CSS (look and feel)
- Fonction de recherche par nom de parc (débutée, mais non terminée (fonctionnel du côté serveur))
Иногда превьюшки появляются прозрачными

Если >>ссылка ссылается на ОП-пост другого треда, который подгружается AJAX'ом, то превьюшка появляется прозрачной.
Mixin session etc

Added the ability to mixin cookie or session support.
Added Caravan from the Last Waltz

Fucking awesome foot kicking action: Van the man!
Twitter Bootstrap

Hola a todos,

Acabo de integrar el Bootstrap de Twitter en la sección de administración.

# Cambios #

* Prácticamente **toda** la UI.
* He añadido un **countdown** en el dashboard para tener visibilidad de para cuándo está programado el *próximo evento*.
* En términos de arquitectura de la información he separado, en los detalles de evento, lo que son los detalles en sí del evento de la propia localización. Ahora hay dos pestañas, la de los detalles y la de la localización mostrando un mapa de Google Maps embebido por un ```iframe```.

Ya me decís que os parece! Un saludo!
Christian.
Fixes for various multibyte string issues
Code cleanup 20120206

This set of commits is strictly cleanup, editing both versions of stanford.profile so that they have the same syntax and formatting, and so that the only differences are qualitative, not formatting.
fix variable name

there is no $memcache variable in this closure
Fix to the #remove_default_constraint method

This fix addresses an issue with the *remove_default_constraint* method in the SchemaStatements module. When a database table has both default constraints and foreign key constraints, the return of both constraint tables (by the **sp_helpconstraint** procedure) break the filtering by constraint name that this method expects to be able to perform successfully. This will break migrations that remove columns with default constraints.

By changing the execution of the stored procedure from **select_all** to **execute_procedure** the results are returned in a way that the filtering works as expected.

Hat-tip to gicappa/activerecord-sqlserver-adapter@88e8c876c08792ac487b6031e12ffc2d981c5e44
added variable resolving for windows
Updated instance variabels

Used $_variable_name for private and $variable_name for public
Changed instance $id getter to being aware of non standard object id's

Right now we're relying on documents having a standard MongoDB object id, but it could also be any other value except an array.

See: http://www.mongodb.org/display/DOCS/Object+IDs
The third argument to both the master and client must be the filename of...

... the sourceAddrFilePath file in order for this example to run.
Remove point about D being easy to implement

@donc [removed a similar statement](https://github.com/D-Programming-Language/d-programming-language.org/pull/70/files#L0L212) from the FAQ awhile back. Might as well change this to match.
Fix "Page wiki" button to have a proper tooltip.

The tip CSS class got left off so the tooltip <span> is just used like any other span making the button huge.
Use new button value naming scheme for all butttons.
Amazingly IE 6 & 8 are more similar in some cases than IE 7 when it comes to rendering buttons.  So during this IE dev session I was targeting IE7 quite a bit. I may have originally attempted to target IE7, then realized I needed 6 as well, so the hacks blended and *+ was the result.  It works as intended for IE6/7, but without knowing the exact target & implications, it's of course better to go with something more understood, in this case, *.
It looks like it's assumed for Galleys and Supplementary Files that $request->getJournal will be appropriate, but that's not trusted for Articles and Issues. Should there be checks for the other types here?
it's possible that a valid filename starts with an arbitrary number of slashes, i.e. 
///usr/src/
Please, consider applying https://github.com/dvv/luvit/commit/00f586a012c9ee4aed562798925c8d0ff747229b.patch -- a fix to allow tls build by standard tools. TIA
@pixeltrix looks nice - thanks!
Sure, that can go...
Now the permutation test is included (could you have a look over it?) and the documentation is updated. P-value adjustment is the next thing to include.
However the permutation test take loooonng (12 Minutes, with the bryceveg-data and 999 permutations).

Should we give the "pull request" another try? Hopefully without any conflicts... If so, I think i have to change something on my worklow (currently I am working directly on my 'master', but I have read that this isn´t the preffered way [https://github.com/sevntu-checkstyle/sevntu.checkstyle/wiki/Fork-and-Pull-Request-workflow].
I haven't found any other to get other clicking on the 'use' tag. i.e. you have to know it exists if you want to find it.
Would you accept a PR mentioning it in this file (just to tell it exists and give a link) ?
True. I thought that Rails 3.1 Engines would bring a that task like they to with db:seed. Seems that's not the case.
Ah, yeah, I've been in that situation, too, where there is a bunch of 3rd party stuff blocking the page load.  It sounds like using domReady as a module is the slightly friendlier way to go, and my only real problem with it is that it doesn't Just Work without a loader config path entry (or putting the loader's domReady module directly in baseUrl).  I need to verify if Dojo provides a domReady module (I hope they don't special case it as a plugin only).  Anyway, I'll ponder that a bit ... maybe it's just a necessary evil.

For now, go with your own domReady, and I'll open an issue.

Go figure, domReady and wire/debug have been the parts of wire that have been the most difficult to get "right", and seem like such simple things :)

Your DOMSubtreeModified idea also sounds pretty interesting ... I need to read up on that.  Def let me know if you come up with something cool!
"Or" is not meaningful here anymore. And there is left a mention of plugins a few lines above.
![](http://bukk.it/wonderwoman.gif)
<3<3<3 :)
Correct. The difference between Artesis 1.2.1 and 1.2.2 are minor (Ding2 version number and two LFs in a module). The major change are in the frontend modules included by Ding2, changes which have been pulled from the frontend modules included by Artesis.
HUGE loss of documentation.
스택오버플로우처럼 Add Comment 버튼 누를때만 코멘트 폼이 보이도록 해봤습니다
I'd been planning on doing this as well, but one thing I wanted to do was to "snap" to keyframes. When it crosses over a keyframe, it actually hits that value instead of interpolating between it and the next frame. With this, if the artist has gone to an extreme value at the keyframe, the animation can skip over it and look off. At least that's the theory.
Not sure how is it related to this, because Arcane Explosion is not applying any aura.
It compiles natively some files in stdlib and in hipe. The compilation happens only when the number of files to analyze is more than 20 or so.
I suggest you do the following:
   ./configure --enable-native-libs
   make
   rm lib/hipe/ebin/erl_*.beam
   make hipe
which will also make two files under hipe with native code. See how long it to dialyze after that.
#144 #172 #118
Я про новые попапы. Изначально как задумывалось? Ещё когда патч в дев лежал.
wrong indentation...
No, probably why it has a TODO on it as well.  

Since this is almost 7 months old.  It's safe to say that he won't be fixing n's dumb prefix detection
Then we should remove validations on namespace ?
so will i then, ill change it
Ooh, cool, I should look at this a bit. I've been [waltzifying](http://dancehacks.tumblr.com/) things for a while, but I've found that you need some different tricks to make things sound better for dancers. I haven't found a good way to backport things to the Remix beat manipulation code, though. Is there a good way to mix audio / overlay beats?
CHTSPData::ParseEvent already destroys msg if the return value is false. XMBC crashes here it this occours.
Why are we using this instead of the functionality in the stats library?
Oh I see. I think inference is the way to go. @WalterBright?
Good coding style, but inform the user what has happened.

```java
if (selection == null)
 return;
 Path path = Paths.get(selection.getData().getFile());
if (!Files.exists(path))
 return;
if (selection.getData().getType().equals(IPatientService.TRAIN))
 return;
```
discussion with lsmith: we just keep it that way for now. but we need to reset those as well
Also about data files, I've seen that `pkg_ressource` is the way to go rather than using __file__ because __file__ will fail il the app is packaged with py2exe (or so i've heard)

http://stackoverflow.com/questions/1711085/introduction-tutorial-to-pkg-resources
We want this fixed, but I'd suggest doing so by creating a write function (in addition to log) that works like log but with no added newline. For node that should correspond to process.stdout.write I think. Possibly with a slicing fallback, depending on whether we want --source to work outside of node.
@peller Didn't you say recently that it was better to use `bind()` (or hitch) instead of variables like this?
Martin said if you do site = getUtility(ISiteRoot) you can't then use site.REQUEST. I haven't double-checked that.
Yup, check out 1282e26a158dbfc2c12392d4bbd6993de1a17cdd
Does `"%d"` ever give different output than `int2str`? I don't mean general %d, but the exact format string `"%d"`.
You could add a simple test in tests/test_commando.py too.
It should be handled. Stopping the analysis means that it is completely aborted.
just for info, there is tool released https://github.com/xfurry/eventAI_tool/tree/master/EventAI%20Creator
l solved my issues, as I realized that my local branch was fully set back to the branch state (in this case 1.6). 
I opened the news file and corrected these errors and commited. Everything fine.

I paste my (slightly commented) bash history here - to show how this happened to work - maybe there is a faster and cleaner solution doing this and someone more skilled in these things can show how to do this any better, please, as this seems quite complicated to me.

 (master)
$ git pull origin master

 (master)
$ git checkout -b 1.6 origin/1.6
Switched to a new branch '1.6'

 (1.6)
$ git cherry-pick "commit-number"

 (1.6)
$ git pull

 (1.6)
$ git push

 (1.6)
$ git checkout origin/master
Checking out files: 100% (835/835), done.
Note: checking out 'origin/master'.

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 6d4bd49... 'the last commit note'


**And here is how to get out the 'detached HEAD' state mode:**

 ((6d4bd49...))
$ git checkout -b temp
Switched to a new branch 'temp'

 (temp)
$ git checkout temp
Already on 'temp'

 (temp)
$ git diff master temp

 (temp)
$ git diff origin/master temp

 (temp)
$ git branch -f master temp

 (temp)
$ git checkout master
Switched to branch 'master'

 (master)
$ git branch -d temp
Deleted branch temp (was 6d4bd49).
I don't think this code is correct. This doesn't update `link->offset` any more (making subsequent `collfs_read()` and others behave incorrectly) and because `fildes` is not a valid unwrapped file descriptor in general.
我开了一堆的窗口去刷票，提示有票的时候，也能否将那个窗口激活啊。
等我手忙脚乱的去点到底哪个窗口刷到票了时，票已经被人抢走了。郁闷啊。
Understood on the knob.  I think the main goal is to keep allocation at the specific multiples so that reuse is easier in the free lists.   I believe jemalloc rounds up object sizes to ensure reusue and prevent fragmentation.
Ah thank you. I was wondering why you said to regex it when I was just changing it in index. 

Sent from my iPhone

On Jan 18, 2012, at 9:02 PM, Divya Manian<reply@reply.github.com> wrote:

> er no we cant change this in index.html. It needs to be either done in app.js or in markdown. I would rather do this in app.js. Doing this now.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/h5bp/html5please/commit/a80b1458ba520cb9b4f251d8eff712b0468385c4#commitcomment-880578
Okay, I've made this explicit.
But... but it might never come back!
Está medio raro esto, mejor algo más legible como:

```ruby
File.join(Rails.root, 'test', 'fixtures', 'files', 'test.pdf')
```
would we still not want this test to make sure its working?
this looks good in and of itself, though I don't have full context of the primary codebase yet. But on its own, this looks right.
damn, eagle eyes.
(edited the previous comment to keep github from thinking that "ed25519" was a git revision id. That's kind of hilarious, actually).

I hear you. I didn't document it, but `create_keypair()` actually takes an `entropy=` argument which defaults to `os.urandom`. And you can create a `SigningKey` with any 32-byte string of your choice. So it's easy to use it in the way you describe.

I'm on the fence about whether to **require** people to use it that way. The original C code sought to prevent people from hurting themselves by hard-coding a read from `/dev/urandom`, and not allowing determinstic key generation at all. I had to change that to let it work on windows (where it still uses os.urandom), and to allow determinstic testing. I could imagine requiring a fixed-length seed as you suggest, although that would take it even further away from the original author's design.
This was because of a compiler warning. Due to the methods/properties being split into two different categories, the compiler didn't know that the getter was being created in the other category. Thus, it gave a warning about a missing getter.
Yes, victim of python interference
The MCNSAPo8 plugin now utilizes every part of the new api.php
word. im waiting on @colegillespie to finish his initial hookup of your recline fork with this branch of datacouch then we'll be in business. https://github.com/maxogden/datacouch/commit/aa64b26d184794e6d8f5a89c01882239f9622be7 added the ability to create the same sandboxed js environment in the browser and on the client to control what modules are available in both places. this means when you do a preview/transform in the browser it will be the same javascript environment as when you do it using the new server side transformation stuff. the API for server side transformations right now is to submit a doc that looks like this: `{ "type": "pendingTransformation", "transform": "function(doc, emit) { }", "dataset": "abcde12345", "user": "maxogden"}` then when it finishes it gets updated with a `finishedAt` timestamp and the `type` gets switched to `transformation`
Кажется, этот коммит ломает компиляцию проекта, откати изменения, пожалуйста!
Thanks for all your commits James.  The RPNCalc is sweet!
Hi Zeno,

I've created a patch for that commit here:
https://gist.github.com/1633833

Cheers,
Clemens
Put in the same check as for the client (check if port is <= 0 or larger than 65535)
I think what may be even rarer is someone using enclosing hash tags as a literal string.
Also note tags with `output="true"` would require interpolating hash tags as well.

I find having cfscript completions inside hash tags where ever they me be extremely useful.
I think I understand now. Thanks.
这种写法太过魔法，以后逻辑再复杂些，容易混乱。
The syntas was like {% if forloop.counter|divisibleby:4 %}, but works well.
Spell 70672 crash server too.

http://pastebin.com/BKnfJc8j
Both fixed in the latest commit. Also updated the size diff.
voilà donc la formule magique !
Cada View usa N elementos visuais, algumas Views usam X, outras usam Y. O carregamento dos estilos será feito dinamicamente, os arquivos serão mesclados, compactados e, então, será feito cache do estilo de cada View. Não vamos juntar tudo, para não ficar carregando lixo desnecessário.

Além disso, cada arquivo representa um elemento específico. Com isso, cada desenvolvedor pode trabalhar em algo simples, pequeno e, mais importante de tudo, entregável.
MKStoreKit is ARC only.
ah. ok :-)
Btw, add yourselves to the authors array so I don't end up adding wrong mails.
@alindeman I've [ported the JSON encoding middleware][1] over to faraday_middleware. It's not released yet but will be before the next Faraday.

[1]: https://github.com/pengwynn/faraday_middleware/commit/198a4571f22730ddad4b05c62e04da0ea4da404c
this seems rather expensive
Looking pretty good.  Can you fix up the commit messages as per the other pull request.
Aligns well with the tabulation.
Зачем нам цикл, если необходим только код последней нажатой клавиши?
FYI, ^:dynamic is fine. ^{ :dynamic true } is just a habit I have, the difference is taht ^{…} form is compatible with Clojure 1.2 but for ClojureWerkz projects we don't support 1.2 anyway.
I believe this changes the behavior for comparators defined as CoffeeScript bound methods, where the arity will always be zero. They'd previously be subject to `sortBy`, but now will be subject to `sort`.
Yeah it's a bit crazy. I'm a chrome user and on my setup works just fine. I guess it's probably too difficult to write a function that fits with the needs of all the possible users of the plugin. Probably, at this point, the best option is just merge this branch and wait for issues. Thank you very much for the help. It helped me to better understand the issue.
Great catch. Perhaps the message could be more verbose? Something like "Import task creation failed for %r"?
Yes, agreed.  I nearly stole your function to move it into jlib.
I think it may be a good idea to have variations (with enough probability that students will actually see them before becoming proficient) like "five and three hundredths" or "four and thirty-six thousandths", where you have zeros right of the decimal point, to avoid students of getting into a pattern here of just writing down the second number given to the right of the point (i.e., 4.3 and 4.36 in the examples above)?

Currently you're not generating such variations at all, so the wrong pattern of answering will always work.
Shouldn't this be:

```DIFF
-    YELL_RANDOM1                                  = 3
+    YELL_RANDOM                                   = 2
```
missing colon (:)
I did a git rm on the deploy folder with this commit because I guess it wasn't removed yet on the issue11 branch.  So hopefully it is gone now; it is no longer in GitHub's view of the project when you use the code browsing
We wanted to change this to something more readable.
Sure thing. http://jonalv.blogspot.com/2012/03/i-just-got-asked-to-explain-my-choice.html
Thanks CBGoodBuddy for the patch.
I can confirm that it resolved my issues with forked-daapd 0.19 on my Ubuntu server 11.10 and iTunes 10.5.3.

One issue I ran into however (surely Ubuntu specific but I'm putting it here in case someone trying to apply the patch on 11.10 runs in the same problem), namely that compiling forked-daapd fails "out of the box". I had to manually reinstall libevent1-dev (upgraded and removed by the upgrade to 11.10) for it to compile without errors.

Thanks so much for your help!
Shame. 

I do hope I said hello to you at the second conj. If not, I'll definitely make up for it if I get to go to the third...
It just moved up a couple lines, so this will look a 'lil bit nicer:

http://underscorejs.org/docs/underscore.html#section-10
Tend to agree with this, but on the fence about removing the similar lines that I stuck in some of my JS micro libs, since they seem to work great. Was taking underscore as a best practice at the time.. Guess it's different here because of underscore's serving as Backbone's dependency.

I'd say if removing this it'd be nice to add a note in the docs about AMD, perhaps linking to some article explaining how to set up that easy shim which would export underscore and Backbone in `noConflict` for when people need it.
**Part** | **Points**
------------- | -------------
base | 1
awesomeness | .25
**Total** | 1.25


Drop the ()s around your conditionals please. Ex:

```python
if (n%3==0):
    print "3.", n+4
```

```python
letters = ["D","C","B","A"]

grade = grade/10

if (grade >= 6):
    print "4.",letters[grade-6]
else:
    print "4. F"
```
Is awesome! This is a very [elegant solution!](https://www.youtube.com/watch?v=QZB7OFRUrA0) [Mighty Boosh dance sequence]
This filter should likely be changed to work on the `show` action.
this fixes both auras.
and spells for:
frostfire orb
ravage
lacerate
insert swarn
eclipse system (druid)
Pyroblast
launch!
todo:
// Drain Life
// Chakra
// Chakra: Serenity
// Soul Swap: Exhale
// Wild Mushrooms detonate

credits for some codes from MigCore and CataProject.
btw thanks swizle for the idea i will look at it. 
we do need someone who can do scripting fo sur'
Sweet! I end up doing this on most projects. Critical if you're migrating from another platform.
Do you think we should stick to a consistant package name? I like org.squidwrench.gameworks.somethingsomething, but I am open to suggestions.
Oh, I see!

While we're on the subject, it would also be nice to be able to declare a transaction as READ WRITE or READ ONLY.
tem como implementar um plugin para wordpress?
Übernommen. Trotzdem ist mir noch nicht ganz klar, was es mit der "Twitter app used for authentication" auf sich hat. Wenn ich es richtig verstehe, kann man in seinem Account mehrere Applikationen anlegen und irgendwie (?) verwenden. Wenn Du Dich damit auskennst, versuche doch bitte, das noch etwas klarer herauszuarbeiten.
This line needs to be uncommented in future revision
happens with me too sometimes, n then this won't open in anything but openoffice
If it only does this in its unittests maybe you could wrap it inside of a version(unittest) section?
Isn't map(&:to_sym) a new 1.9 syntax?
I've changed the name to isRecurring, but I guess I didn't get that pushed. The recurring data is in a Recur object that goes with a RecurringInterface on the product.

I think it is overkill to create a new class just to ask if an item is recurring or not. Its a fair question to ask and isn't tied to a service or good. A magazine is a good and it is recurring. A service does not have to be recurring. I can have both good and services in the same cart and I can have both billed once and recurring in the same cart.

 If you don't put some type test like isRecurring as you go through the items in your cart during checkout, which you must do because they are handled differently on billing, then you have to test for the instance of the said CartItemService. IMHO, way too much complication. It is the equivalent of having a different cart item interface for a free product.
I simply made the fixes in a new commit, although I didn't rollback the resp -> response changes. Just switched to returning response rather than null if parseBeforeLocalSave is undefined. Haven't tested, so let me know how it looks.

I decided to use https://github.com/janmonschke/backbone-couchdb - CouchDB handles two-way replication and can be installed on a mobile device. It also has more storage space than localStorage, so it fits my project requirements much better.
Separates concerns.  Finding negative numbers isn't part of #add(ing).
1번라인 코드라인에 직접적으로 코멘트를 달수있음
Do this case first so can reduce nesting

if (qbItem == null) {
  return false;
}
productId = ...
honnêtement, c'est vraiment inutile et ignoré par le navigateur, hein :-)
null and be in the future => null and will be in the future
The app is currently prepared for the release. I hope that it will be available in the store within the next few weeks...
Really dude? Use every last line of custom code I write, but take the time to pull my name from the copyright. Classy...
Put the LeftMessageTray@robytrevi.net folder containing the extension.js and metadata.json files in .local/share/gnome-shell/extensions folder, replace the shell and activate the extension.
You are right. I'll change this.
oh, just noticed this.   this must be an accidental carry-over from a test you copied.
no mocking needed in this test.
:+1: to "row" and "col". That makes it easier to visualize what the names are for, which in turn makes it easier to remember how to use them.
We can use findNodeByProperty to do the job.
Just for better debugging and troubleshooting purposes, can we split the logging instances into two conditions? one for the null check and one for the isEnabled() check with each having specific error messages?
this javadoc is redundant
s/may/will/g
> I think we should enforce this (the responsibility-uniqueness) when  
> things are put into the transportMapping and not per inbound message.

It cannot be done easily. The problem is that the address part after the  
scheme string in the URL may have transport dependent meaning, so only the  
specific transports know if their responsibilites overlap. For IP based  
things subnet masks are the most obvious filters, but who knows what kind  
of filters people want.
You can remove them.
you know this is done for each `@Test`?
we have used BeforeAll and a static system variable at some other places
I don't think "nextClockwise" needs to be a method. Also, try to make it return a single value (not a tuple).
ok, I'll add a guard, akka is efficient but running on 0 bytes max heap is in your dreams ;-)
A warning would be good.  Given the technical audience of boto and the fact that this wouldn't come into play unless someone added a keyring option to their configuration, I thought the import error would be pretty clear, but I'm happy to add a try/catch with a message.

Whether you add a dependency is up to you.  Lots of people get cranky about additional dependencies.

Let me know which you prefer.
Nope, you have to think about it more.  set_attributes hasn't run yet, so the underlying card is still in the 'pre-save' state  This stuff it hit in before_save filters.  Go ahead, try to take it out and see what happens.
The above change where WikiReference becomes Card::Reference, but it must be right.  Seems like an odd move, but I think I remember now.
Okay, thanks. I'll update that.
%r = repr(arg)
Ah, how complicatedly nice.

So, who is who?
There are several user ids and user objects and user dicts within a controller action in context, in c and in data_dict. I can't figure out what is for what.

I guess that the user in context is the one who is has called the controller action. So, why bothering with authentication when this is already done somewhere before the controller action call?
@tobes 

I think the use of "deprecate" in the name is correct: the item is still available on the context object.  But accessing it logs a warning that it will be removed in some future release: just as the `@deprecated` decorator above does.

On an a related note (but not related to this pull request! ;-) ) I like the idea of a sister-function that does what you describe: indicate to the user that this item is no longer available in the context by logging an error and throwing an exception.  I don't really like the fact that if you access an attribute on the global context that doesn't exist, then it simply returns an empty string.  So the use-case for this sister function would be in v1.9 we would call `removed_context_item('facets')`, and that way the user gets an early warning that the context no longer carries the `facet` attribute.  Then in v.1.10 we would remove that call as users have had a release to sort their issues out.  What do you think?
I think we want to add a configuration section that talks about it, so when we do we can circle back and add a link to that section here.
Couldn't stop picking on this 'til it was done.  May be some needed tweaking and fixing, but mostly it seems to be very clean and elegant now.  Most of the stuff ends up in card, lib/wagn/model/reference.rb
Why this? Why can't start time and stop time be set properly instead?
thanks, I'll give that a crack.
I'd prefer progressively increasing timeouts (up to a maximum, say 30s) - rather than timing out and having to restart the server.
From a design standpoint it's pretty marginal either way, I think. I like having the getter abstraction there just in case, but it's pretty unlikely something would change.
If this manager works exclusively with S3, might want to call it `S3UploadDirManager`. If not, never mind - I just have more code to read.
Same here, just `return` from within each `if`/`elif`
2012/6/29 Krzysiek Goj
<reply@reply.github.com>:
>> @@ -40,6 +43,10 @@ connect(Args) ->
>>      Transport = gen_server:call(Pid, get_transport),
>>      {ok, Transport}.
>>
>> +send(#transport{socket = Socket, ssl = true}, Elem) ->
>
> How about compression & SSL?

TLS already supports compression, so stream compression using other
methods (e.g. zlib) is intended for situations where TLS is not to be
used.
Hey where's this guy going? Tie that mark down down... =)
You are completely right. I have moved the prefix out of the setStyle.
Thanks for contributing to tiled. :)

> I am not a big fan of C++ nor Qt, but I must admit that this project is one (if not THE one) most well coded C++ project I have seen in a while!

To keep it this way, please adapt the coding style used as in the project before.
Opening brackets ```{``` go in the same line as the if, while, for, except on function declarations.
senderOption =
      if (!msgPdu.hasSender) None
      else ...
Same answer as above. :trollface:
These three methods are unnecessary.
Should be `nsnotify` instead of `nsnotify_guard` (and also at line 11 & 18).
Style issue: space between if and parenthesis. Happens again a few lines below.

Another style issue is that code examples should use two space indent.
change to self. observers should access the interface for content and method.
I think that is not correct
I think we can just remove it..


Am 01.10.2012 um 10:42 schrieb Jestan Nirojan <notifications@github.com>:

> In codec/src/main/java/io/netty/handler/codec/sctp/SctpMessageToMessageEncoder.java:
> 
> > +
> > +package io.netty.handler.codec.sctp;
> > +
> > +import io.netty.buffer.ByteBuf;
> > +import io.netty.channel.ChannelHandlerContext;
> > +import io.netty.channel.socket.SctpMessage;
> > +import io.netty.handler.codec.MessageToMessageEncoder;
> > +
> > +public abstract class SctpMessageToMessageEncoder<I> extends MessageToMessageEncoder<I, SctpMessage> {
> > +
> > +    /**
> > +     * Returns {@code true} if and only if the specified message can be encoded by this encoder.
> > +     *
> > +     * @param msg the message
> > +     */
> > +    public boolean isEncodable(Object msg) throws Exception {
> What do you think, is it worth to have this or shall we remove this class?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
`fetchOne` should work fine, since this is a `SELECT COUNT`.
фигурная скобка должна быть на той же строке, что и оператор (то же самое и ниже)
непонятное имя, лучше переименовать во что-то типа saveConfiguration()
передвинуть вниз
Strongly strongly strongly disagree.
Use the string interpolation.
SCHEMA was already being ignored.

I'd like to ignore Explain as well because when a query triggers the auto explain by passing the threshold, the original query and the query plan will be logged. It's not that interesting to see the Explain query logged as well.

By itself this isn't an issue, but there are other debugging gems that hook into ActiveRecord to force explain plans on every query which ends up logging quite a bit of useless queries.
THIS. IS. LIBRARY.
Its hard to imagine that most of these symbols need to be on this list.  I know it was taken wholesale from the list of symbols forced at the end of definitions, but that was method-private.  Suddenly it's the canonical list of "magic" symbols.
To be honest, I don't feel like the absense of indentation is justified here.
The problem previously in `Wrappers` was that the byteswap-hash magic number constant in `improve` was missing a digit - the hash worked ok still, but with a lower chi-square for this test-case.
When I refactored and fixed the missing digit, chi-square increased slightly. Currently it's 2.7, which is still pretty ok.
At https://github.com/mattpap/sympy/tree/unified-gcd you will find how `gcd()` and friends should be implemented. Now you can change gcd and lcm definitions in appropriate domains (for floats it will be RealDomain (which should be renamed btw.)). I may add another commit there, removing gcd and others from core (not igcd, ...) and use igcd and others internally in domains.
```
In [42]: gcd(S(3)/4, S(1)/2)
Out[42]: 1/4

In [43]: gcd(S(3)/4*x, S(1)/2*x)
Out[43]: x
```
This is correct because gcd, lcm and others always give monic results over a field.
Why was this change made? Can you evalf to a lower precision, say 2, and still get the answer?
delete this function since you have it as a method of Matrix above.
May I complain, once again, about the use of dark magic and global state in docstrings?
done


On Fri, Nov 16, 2012 at 2:02 PM, Ronan Lamy <notifications@github.com>wrote:

> In sympy/unify/usympy.py:
>
> > @@ -0,0 +1,109 @@
> > +from sympy import Basic, Expr, Tuple, Add, Mul, Pow, FiniteSet
> > +from sympy import Wild as ExprWild
> > +
> > +from core import Compound, Variable
> > +import core
> > +
> > +class Wild(object):
> > +    def __init__(self, arg):
> > +        self.arg = arg
> > +    def __eq__(self, other):
> > +        return type(self) == type(other) and self.arg == other.arg
> > +
> > +def sympy_associative(op):
> > +    from sympy import MatAdd, MatMul, Union, Intersection, FiniteSet
>
> is there any reason to import these at function level?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/sympy/sympy/pull/1633/files#r2156231>.
>
>
This tests nothing.  If you want to test that it remains unevaluated for symbolic input, you'll have to structure the test a little differently.
all this can be written like:

```python
ke_sys += e.kinetic_energy(frame)
```
No extra variable needed.

Check PEP8 too, caps in variable names is no recommended.
Why not just `__init__`?
Definitely `course_membership_test`. And I don't think we need all the `Couse.new` and `course.course_memberships.build` boilerplate. Since we're only testing `#instructor?` we could do:

```ruby
@membership = CourseMembership.new(role: "student")
@membership.instructor?.must_equal false
```
s/there one/there is one/
It looks like incompleteopus added this, and it doesn't look current because all of the tests are passing. I'll remove it.
TreeLoger isn't needed as a parameter
Commons logging
Hier auch den eingeloggte User verwenden.
With the suggested switch to this being an option, this would have to be `s.initiatedBy.getOrElse(null)`
I don't think we should necessarily limit rules to only intervals.  Let's pass in the whole DataSegment and let it make a decision based on that.
I don't think the phrase 'context for your session' is quite right.  I know we hate passive voice, but if we're keeping the sentence structure, I would go with 'is selected for your session'
"that resembles the following"
I'm curious if there's a reason you don't check for the ::PhusionPassenger constant here like you do in local_environment.rb:400.

Thanks.
Remove the `!` and swap the statements.
All js should be included in the footer. Includejs was created for one use case, there are other use cases that I'm sure it does not support. The correct solution probably will be altering includejs or adding similar tags for other use cases (inline js, js files referenced with absolute paths, etc.).
I thought without backticks the selector was too indistinguishable from the rest of the message.
Numeration is broken
Test should not include another test. This code should be in a helper or base class.
Sunsilver2 hasn't got any AJAX stuff yet, so of it doesn't have jQuery it
should be fine to leave.


On 26 Jun 2012, at 04:12, "David Colón" <
reply@reply.github.com>
wrote:

@@ -64,9 +64,46 @@

               }

           }

       }

+<!-- ENDIF -->

+

+        /**

+        * Toggle a section's visibility status

+        */

+        function toggle_section(link, panel_id, expand_text, collapse_text)

+        {

+            if (window.jQuery)


The problem here is that this is subsilver2 and there's really no sort of
ajax in subsilver2.  So either I create an ajax.js for subsilver2 or just
remove all subsilver2 edits.  Question is do we want to pursue this at all
with subsilver2 especially with its imminent retirement?  core.js is not
even being included in the footer like it is with prosilver.

---
Reply to this email directly or view it on GitHub:
https://github.com/phpbb/phpbb3/pull/768/files#r1048027
sorry, misread how the uid works.
I don't think there is value in using type here. All the cases I observed use 'nil', but it seems like a good way to shoot yourself in the foot: I declare 'i32' but mistakenly pass in a string.
wont' this mean the event loop will never exit due to zero pending things?

Seems like we want a 'weak' timer :-/
The way I've written it does use the database. The query is:
SELECT "activity_activity"."id", "activity_activity"."timestamp", "activity_activity"."user_from_id", "activity_activity"."user_to_id", "activity_activity"."message_string", "activity_activity"."arguments", "activity_activity"."game_id", "activity_activity"."public", "activity_activity"."action" FROM "activity_activity" WHERE ("activity_activity"."action" = seen  AND "activity_activity"."user_to_id" = 1 ) ORDER BY "activity_activity"."timestamp" ASC LIMIT 1

This makes the scanning stop after finding the first element. Since there is an index in the database this is most likely O(1) complexity.
This block looks a little confusing to me. I could understand what it does after looking at the `play` and `reset` methods, but I think it’d be much clearer if it looked something like this:  

```js
this.rewind();
this.repeat -= 1;
if (this.repeat < 0) {
  this.reset();
  this.emit('end', this);
}
```

with `rewind()` and `reset()` like
```js
rewind: function() {
  this.frame = this.currentTweenIndex = this.prevFrame = 0;
  this.currentDelay = this.delay;
  return this;
},

reset: function() {
  this.rewind();
  this.isPlaying = false;
  this.clock.removeListener(this.isTimelineBound ? 'advance' : 'tick', this, this._onStep);
  return this;
}
```

`reset`/`rewind` could then also be used in the constructor to remove some duplication
`this._config.appConfig` contains the appConfig overrides that were sent to the ResourceStore constructor. I believe that (at least) one command is using this.
Punting on this micro-optimization.  This happens at server-start time, which isn't time sensitive.
No, default you don't need to do anything. But when deploying to heroku you can set a environment variable to overwrite which version of python to use. Since you need to run python 2.6 on Heroku to make this work.

`$ heroku config:add GLORIFY_PYTHON=python2.6`
Still, a view model is not a view script. But since this is logic meant for output purposes, it should be located in the view layer (= view model in this instance I reckon)
Please remove comments if not needed.
More copy'n'paste carnage.
Lack of docs, so it's not clear if this takes a filename or a file object or either.
@richh98 Is this ok?
Add an error msg.
Does supporting both sql.Date and util.Date here unnecessarily add methods?
Maybe the simple thing to do is to support getTimestampAsDate and return a constructed sql.Date with a util.Date return type. That would add two fewer methods and make the method names simpler.
Non-unique index support is added later after this comment, I should finish this comment after I finish the support for non-unique counting index feature. My fault, thanks paul.
shadowing gettext?
`promise = exports.readJsonJs(includePath);`
mentioned in the below comment.
The type name should be capitalized.
Thinking ahead a bit, calling jQuery.ajax directly is probably not a good idea here. If we let people drop in alternate ajax implementations they may not take the same args. What if we add a simple API to ajax.js that does this, and people can shim in their own if they replace jQuery.ajax?
So, turns out the most concise way to do indirect eval is `( 0,eval )(src)`. @dcherman says so!
"unqualified"?
Y U NO CLOSE? xD
I believe this code has changed on auth in the meanwhile, right?
I have moved these lines down below to the Linux part of the if branch.
thanks. Fixed. Push later.
Let's add the name of the .repo file the consumer drops our repos in when we mention it's a normal yum repository.
You're talking about legacy code, it's been agreed upon in the team to discontinue the practice of adding both since they have the same meaning, @daniel-uzunu can confirm.
You should make this task async, similar with the 2 tasks above.
die ganze string umwandlung ist nur für den stack erforderlich in dem ich checke was wann registriert worden is.. wg. zu spät und so. Im Firephp log selbst bleiben die callables so wie sie sind.. also alles was kein string is eben anklickbar.. -> viewer
wg. $user: gut, wenn der dort auch avail ist.. andereseits: in der config will man eh die erste firephp instanz initiieren und die options setzen.. d.h. der code bliebe eh, und die eine zeile weniger bringt einem auch nix.. oder?
are you sure the component type's 'toString()' is going to be reasonable? What about exotic types like ArrayList<String>[]
Comment removed anyway.
Whoa, NIO? On purpose? Crazy world.
I'd actually prefer to *not* output anything if the file is missing instead of using "special keywords". The usual advantage: Be consistent (that's the usual behaviour of explorers -- this needs to be documented somewhere).
It seems safer to maintain the original query here. For example, suppose there's another user handler in place that performs some custom authentication based on the URI query, the redirect would strip these away.

i.e. this request:
  /foo?key=123 
should redirect to:
  /foo/?key=123
Is this actually a thing? I was never able to find any references to memory leaks from `delete`.
Maybe `support@singly.com`?
Seems simpler to just return `pendingPromise` directly.
Yes, I'll add an anchor and a draft of installing multiuser Chrome in that page. You may need to correct it after I update that page.
Function trim() used to detect "only whitesapce" is inefficient because it not only checks for whitespace, but it creates a new string without the whitespace. You can use a RegExp to check for non-whitespace without creating a new string, and copying non-whitespace part of old string to it.

Replace this ``ctx.token.string.trim().length === 0`` with ``!ctx.token.string.match(/\S/)``.

There are many other calls to trim() that also need to be fixed.
Hmm, this gets tricky here. What if you have an html file with javascript inside it and the whole file is selected? That would return html (even though the selection contains javascript), right? It may be safer to make this function return the mode for a pos rather than the entire selection.
This isn't relevant to the unit test itself, but looking at these strings, I have some suggestions for rewording--in order of the ones listed above:
* The ftp protocol isn't supported by Live Preview&mdash;please use http or https.
* The base URL can't contain search parameters like "?id=123".
* The base URL can't contain hashes like "#anchor1".
* Special characters like "<" must be %-encoded.
Now code is there twice, rather do something like:

```
var value = e.model.queryValue("general/keybindings/@preset") 
if (!value || value === "auto") {
    ...
}
```
The BinnedData subclass of QuasarCorrelationData already has arrays filled with loglam, sep, z that this _llLookup array would be duplicating. Depending on the access patterns, this caching strategy might actually be slower than the original code (not that it matters).
I guess you are right.  I simply followed the way a similar issue in scala-mode was fixed in #949.  I'll change `:build/darwin` to `:build`.  How does it sound?
Adam, I implemented this, same way as in the screensaver plugin. I'm not to fond of this solutuion though.There seems to be the race condition with initialising objects. I do not have time to really investigate and debug this so for now I have found a workaround and it works well. I will look into it more in the future though.
Please add a comment to explain what this variable is for, since the name doesn't really indicate much.
Do you mind renaming all of these files and variables to `-editor` and `___Editor` respectively? It's not a view anymore, right? It's very much an editor.
Sorry, I assumed the border would be in `%` but it's not ATM. This is fine as is.
This could do with a mixin, right?
Can these be removed too?
Any reason you aren't using `require.js`?