labofsahil/github-event-dataset-2012 huggingface dataset:
Sorry, I never ask again. :-)
We need to rename Wakka to Wikka in LuMiCula, too. :-)
ну зашел я сейчас в ИК. нет никаких крашей (можете сами сходить на тестовом). мб ммапсы там не запрещены? так с ммапсами там много чего веселого будет...
its my commits, unknow because my git config wasnt configured(pc formated)
you could do `self.get_webapp()` but whatevs
Yup, I was trying random methods from the jungle myself too.

Good news is that it works with `YZX` :)
This just seemed cleaner than pulling out parts of the strings with `split`. Up to you if you thing RegExes are too complicated for this part of the tutorial.
When arkania goes online again.
I would support Sbte's suggestion to keep binary files out of the language statistics.
By moving the bbUI.js and css files up a level it makes it so that you cannot just package up the sample directory as an application.  This is why I always made a copy of the packaged bbUI and placed it in the sample directory.  I wanted it to be a simple zip and package of the directory for a running app.
so what the point of implementing such if libs not supported alrdy for 2y? bring something new :)
Где можно прочитать что означают каждые столбики в instance_encounters
(300,1,58630,210, 'Mal''ganis'),
не понимать что означает 58630 и 210.. 
Не засчитывает прогресс прохождения..
С нулём кикает игроков  переодически с сервера, особенно тех, кто в рейдах.
thx rsa ( i hope i have understand the meaning from google translate)
@rsa i used mangosR2 :P why you think i'm used very outaded source?
это фризы от коммита 1631, я пока все его никак не поправлю нормально... можно просто удалить пока.
все группы пропадут и бинд к инстам тоже. а больше ничего.
ask rsa https://github.com/mangosR2/mangos/commit/3bf48c9324c24a9db19494ea79cec6b80df7c351
;)
Just switching it to a method that follows the create rule[1] because otherwise I'd probably forget to release it. It's not really clear how objects should return Core Foundation types they don't own. I first looked at toll-free bridging, but Apple has that locked down.

The *best* solution might be to refactor so NSUUID holds on to a CFUUIDRef, not CFUUIDBytes, so there's no question of ownership.

[1]: https://developer.apple.com/library/mac/#documentation/CoreFOundation/Conceptual/CFMemoryMgmt/Concepts/Ownership.html
Thanks.  Added in https://github.com/mozilla/zamboni/commit/2cb73355dd3038b1e65d7c1df48191fa6e5e71ce#L1R46
Perrrrrrrrfecto Arcane07,  soy español. Un placer.
Everything like this is done in the class Finger. There are also prepared methods for moving with finger that looks clearer. So soon T.finger will be rewritten by sth like Finger.moveDown(); or Finger.moveTo(); . I think it is little useless to use method goTO, because Finger remembers the node it points at, so it has the coordinates from this node.
hi there. a small typo here i think. In this function we are usign variables starting with "contect"... shouldn't them be "content"? so, $contentClassAttributeID

let me know if you want me to add an issue for this and fix it :)

cheers.
As I look back on the flexbox of future and past, caniuse.com is wrong. There's going to be the two different implementations: `display: box` and `display: flexbox`. I wonder if it's even useful to display the old "flex-box" (`display: box`) spec.
wowhead says this spell is leap, used by arelas brightstar in toc5.
Here's a 3rd reference:

http://www.ccim.org/cgi-user/bible/ob?version=lb5&book=act&chapter=7

Again, the two verses are merged.
Ah I see, then the test should be modified to inform the reviewer to attempt to print/print preview without error. Also for consistency with the current code the var `er` should be renamed to `e`.
no, the catch is this output from the textile help (actually i'd need to catch false positives for art_id 5 as well).. see:
https://github.com/redaxo/redaxo4/blob/master/redaxo/include/addons/textile/functions/function_help.inc.php#L157-158

so whenever textile help is shown - e.g. if you edit a block where the module shows the help, and my textile modules do - those links will be interpreted via the rex_replace_links() and further rex_getUrl.. and if then art_id 7 (appies to 5 as well) is _not_ availlable, the RexSEO warning would be technically approriate, but missleading and annoying..

However good you asked, since i realize i need to extend the checking to art_id 5 as well, plus not try to match the whole string, as it's lang dependend..
Ça commence très fort!
http://svn.mozilla.org/projects/l10n-misc/trunk/browserid/README - workflow

MPL2 and GPL - I am not a lawyer, also I prefer to not abreviate that sentence ;)

Ya, let's ask gerv.
You should push this to  your main branch. Only the tex file though. not all of the aux, etc. files.
I dont think we need to support anything earlier than IE8, so lets not claim this here.
Also, what about android and silk browsers? We can claim to add support for those in the near future.

It would also be good to have some position on how we compliment (or don't conflict with) other popular libraries like jQuery, KO and Prototype. Even if we aren;t doing anything specifically, making a thoughtful statement on our position on these will help establish a path for us to take around these other libs.
I see your patch, thanks!
This won't work with the object cache pattern. All the callbackCache patterns try to store an array in cache. Try the follwing: https://gist.github.com/aca19f8636519976a52e
Regarding roles it should be [the roles setup in ding_default_permissions](https://github.com/dingproject/ding/blob/master/ding_default_permissions/ding_default_permissions.features.user_role.inc).
I agree. I will make the changes.
Cool, this is exactly what I was thinking would be the best approach at least for now.
Why the hell did you change everything from Def to Definition?  It's quite obvious what Def is, especially seeing it's in the definition package now, and it's a royal pain in the ass to type Definition every time...
This spell is trigered by mounts with TARGET_95, so targets the rider. But this spell itself has TARGET_95 too, so when you try casting any spell that trigger this spell will always return invalid target.
@IBiryukov How can owner be null if this is the ID of the entity?
Thanks! These kind of tests are exactly the ones I meant.

For the X stuff: If it's not possible with xclip/xsel, it will be loads of work. I once tried a direct implementation, but quickly gave up (also see https://github.com/kfish/xsel/blob/master/rant.txt ;) ). I am not really into it, anymore, but if someone is trying it, I would try to contribute/help/review.
Writing too much data to the terminal at once causes EAGAIN errors. Writing it in 100-byte chunks seems to get rid of them. There's probably a better way to figure out how much the terminal can tolerate.
Ahhh gotcha - I thought that was what you were going for.  Is there any way to specifically test that the `on_failure_lock` gets called when a Resque::Dirty exit occurs?
This damnable language
Yeah, this ultimately has to do with the historical evolution of the data format (aka laziness on my part)... We should change it, though it'll also mean changing `website/sample-tracking-info.json`, which is used by the demo.
i wouldn't waste your time looking over the differences in bin/lumbar. i changed the whole file for the most part. i added two new args, --include and --use that mimic the functionality found in stylus. i also removed the dependency and copied the style of parsing command line args from stylus. felt it was very clean and easily understandable.
I'm sorry, I added it to that issue since you told me on the earlier commit to alphabitize it.
Checked, and this class is not used anywhere in the AS codebase itself.
Changed it here: https://github.com/Akihabara/akihabara/commit/e8acb58b8c6b0f90a3796266dd0657d5e69eb2e7 :)
You could save one texture2D call here by using the textureColor value.
for what we need this?
given that when acquiring the lock we did try to load the node, this will probably work right now, but just for consistency i would use session->nodeExists instead
_Ough_!
Si, tengo que ver el tema de los SQL en cuanto tenga tiempo. Cuando publiques eso lo checkeo y mergeo ;-)
Process next tick has the same problem as setTimeout is trying to fix.  The reason for using setTimeout is that the callback is called from C++ code which can not be overwritten.  I am thinking that a simple C++ module might fix this issue by simply calling back into a function and preventing the system from figuring out who the parent caller is.  

https://github.com/joyent/node/blob/master/src/node.js#L192
Facepalm. Thanks for pointing it out. Fixed in 3344e2ebdc7da1a25ff64ad5702052de4fe0f723.
That's for sure good to know but who is using it ? That's more something that could be put in a user interface in DSM.
I think we have to keep things simple, if ppl change the deep SPK configuration they change it everywhere including here.
nice clean up
Ha! Forgot to add imports!
Why are you doing the check on the first letter? Just put the keywords in one big array, and do this loop once, unconditionally. It'll only match stuff that matches a keyword anyway. Alternatively, us an object to store the keywords, so that lookup is super-fast. For example

    var keywords = {Call: true, Case: true, CDate: true, /* .... */ VbTab: true};
    /* ... */
    if (keywords.hasOwnProperty(strToken)) return "keyword";
Line 46 in resampler.js should have not existed, was a regression from the last commit and should have been removed there, due to the update changing the algo.
>I mean, I still debate that if the only time you're going to call git_threads_shutdown is right before exit, it's kind of like worrying whether your house is clean before the construction company shows up to tear it down

That is hilarious! New favorite quote.
I added the try finally because it will free resources. I just looked in the DefaultProducerTemplate to check, it stops a producerCache and shutsdown an executor of its own. the context stops a whole bunch of other services.
i thought you were claiming the second was a solution for the first. Anyway, punting on that particular bug, I don't think we're going to be sending non ascii unicode characters as api parameters anytime soon.
Carsten-

Thanks for this.  FYI, I think version.php still says 1.92 RC4
Actually that pointer was removed in the last commit ("Cleaning my act up")

On 07/03/2012 14:22, Dent Earl wrote:
> I think you'll want a space after "system" here, as in
> <code>logger.info("Using the grid engine machine batch system " + str(batchSystem))</code>,
> yeah?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/benedictpaten/jobTree/commit/376ada010e628ae34970687325b820aa7582751d#commitcomment-1055954
Interesting question. I don't think this usage needs any session variables, but other uses of ajtooltip.js might, and if so the restoreSession() needs to happen in there (in the ttMake() function).  Guess I'll add one there to be safe.  Thanks.
Johnny, when you have a chance, please review this commit. I have started to restructure the project to define 'Channels' and 'Endpoints'; and I am mostly happy with how things are turning out. The code is still very dirty; but I think feedback would be good before I spend time refining and polishing anything. The one part that feels a little kludgey to me is the channel interfaces (i.e., ISendMessages, IReceiveMessages, IPublishMessages etc). 

Any feedback appreciated.
Your lines are too long, it makes reading the diff hard :)
I noticed that, I just wanted to test making it rotate like moving solids do, but then I realized it wouldn't work right and deleted all the code except for this, incase I wanted to try to use it for something else later. Also, Ctrl+Alt+[ or ] would have been fine with me to use for rotating the brush, that's what I was planning on doing.
You wouldn't use it over >> (Sign-propagating right shift) because, as the name suggests, it propagates the sign, so `-1 >> 0` is still `-1`. The Zero Fill shift un-signs the int, so `-1 >>> 0` is `0`.

The concern of using these operators to coerce numbers into something you want, is that problems can happen by overflowing (so passing it a larger value than 4294967295), for example `4294967296 >>> 0` returns 0. This may have some very undesired effects depending on the application. So be wary!
I'll move this to a new branch and maybe migrate it to a plugin, when You'll be ready to provide some more details. You're implementing this as observer pattern?
Is there a way for fade in only the images just downloaded and not for those pulled from cache?
w00tstrap!
You should check for a non-empty value, not "on".  It varies
This patch works great in Windows! You might need to lowercase the `extension` in the `if` statement as some fonts have `.TTF` extensions.
> fair enough. You are saying autoboxing is a big overhead? Would JIT not take care of this?
Possibly, but it's a hard optimization, it might take a while before it reaches for this decisions and it might delay other nice optimisations.
I also think it requires the  DoEscapeAnalysis=true option, and it doesn't work around interfaces as in our case.
I added some inline comments above, hopefully you can see them.
Shouldn't you move the `var`s on top? if you want to optimize even this just declare `old` before (how actually JS does by itself) and leave here `old = ...`;
All in all, I think your code isn't too bad. Your style though needs a lot of improvement :) Please try to respect common lisp style used in all my lisp files. That's something really important.

Notes on the organization of the source code: The process modules should not contain any user code such as: move to a specific location. This belongs to the high-level executive. Please create a sandbox package similar to cram_pr2_executive where you can put your high-level code in. Please make your open and close functions more general, i.e. they should get a side and a pose parameter. Please also have a look at the designator.lisp file. Replace the container-open and container-closed action handlers in process-module.lisp with your code for opening and closing the container. Best if you learn about macros and how macro expansion can be done in slime (C-c C-m) to understand the def-action-handler macro.
Array.new(size, Byte.zero) is more readable
Also, remove the parens from the outside call
>Also, don't let your branch fall behind the main develop too much, especially with @rapodaca also tinkering with UI stuff - don't want to risk conflicts from the two of you ending up diverging too much :-)

Exactly! How could I foget about it! Thanks for reminding me about it =)
asWidget initialization should be lazy.
Looks good so far. I've asked  a few questions
La encryption key cambiala por:
hrZ2wzs9p7WXI5Jt2HfHoeep8kUcdPQE
Para más seguridad en la encriptación de la sesión...
Awesomeeeeeeeeeee! Great Work!!!!! Tnx you!
（╯°□°）╯︵ ┻━┻ 
And I just merged the earlier revision!
oic. Never mind then.
alrighty then! thx :)
Thanks Omer, will take a look next time I work on this, and I assume prob = Liblinear::Parameter.new sbould be prob = Liblinear::Problem.new in your change.
one suggestion i have is that it seems you keep ascii/pasta in memory... why not make a "dump_file" sub that reads+dumps?  since its rate throttled anyway, having to read it from disk each dump isn't a huge penalty and will cut down on the bot's memory footprint.

    sub dump_ascii {
      my ($file) = @_;
      open(F, "<".$file);
      while(<F>) print $irc $_;
     }

or whatever you do to print/dump text to a channel.  Then in your command handling, just do if(/reason to interject/) dump_file("interject.txt");
Here, the self passed as an argument in the las parenthesis, was causing an error.
This is already the model. SInce selected_field is actually a backbone model of the griditem.

See https://github.com/DracoBlue/sudoku-android/blob/master/dummy/static/js/PlayingFieldGridItemView.js#L33 where the event is fired and in https://github.com/DracoBlue/sudoku-android/blob/master/dummy/static/js/ChooseColorOverlay.js#L28 it is fetched.
This stuff makes munin-node-configure return "non zero value" error
Comenting it doesnt help either
Tried adding some echo before exit , but then the error changes to "Junk printed to stderr"
```
$blah=explode("§e", $Data;);
foreach ($blah as $value){
echo $value."<br>";
}
```
/b2m/password-supply
How did you test this?

I get a error from gdb 7.3:
<pre>
GNU DDD 3.3.11 (i486-pc-linux-gnu), by Dorothea Lütkehaus and Andreas Zeller.
Copyright © 1995-1999 Technische Universität Braunschweig, Germany.
Copyright © 1999-2001 Universität Passau, Germany.
Copyright © 2001 Universität des Saarlandes, Germany.
Copyright © 2001-2004 Free Software Foundation, Inc.
Reading symbols from /home/cj/stm32/FunTechCortexMX_test/main.elf...done.
Remote debugging using localhost:3333
Remote 'g' packet reply is too long: fc000020d4400008000000002300000000200240ffffff7fffffffff404d0020444f0020f77feffbf85e3d48dc96a73a744f0020fc4f0020ffffffff390e00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000020000001
(gdb) 
</pre>
won't this cause a NullRef exception if pSample=NULL?
'll have skyfire skype? I would like you to tell me which is the project that is stable to slightly unstable take much to help
Yep thats true, but you shifted in wrong way, I think ;) (should be ">>")
Why not also upload the RPM package?

BTW, the current package seems dynamically linked to libc. Should we change that by adding a CMake variable to allow static linking to standard libraries
Looking at the differences on the spec:

    http://tools.ietf.org/rfcdiff?difftype=--hwdiff&url2=rfc6455

I can see additional status codes on section 7.4.1 which could be added to the library but I think they are more for giving extra helpful information of the reasons for closing than anything else. I'm not an expert, but it looks to me that you can just return status 1002 and it'd be fine.
It's true that what you've written is the more conventional way of doing this. But I find it aesthethically ugly, and the two are functionally equivalent. I've managed to convince quite a few Ruby folks (beginners and strong developers alike) that my approach is prettier :)

But beyond aesthetics, is there a reason why you'd want to change this back?
This is actually a feature, EPEL must not conflict with RHEL, so python-webob1.0 installs newer version in parallel.
/usr/share/doc/python-webob1.0-1.0.8/README.Fedora describes how to use it and EPEL openstack-* packages are patched that way.
This of course doesn't work when running from git like devstack does, so your symlink workaround is fine I guess.
The 'a' is still required for the script to work correctly. There no longer is a version 7.2.
I can confirm the executable produced functions as described in the README on my Mac running OSX 10.6.8 with the changes performed here. Are you able to merge these in and still have Lion function?
Why static? Makes it that much harder to mock + service methods don't tend to be static anyway.
Should be a input type password.
This might work, but  in 1 test file, you probably need to choose either ```Catalyst::Test``` or ```Test::WWW::Mechanize::Catalyst``` for optimal usage because they do almost the same task.
add updated URL
I'll merge by hand this time, but this won't merge cleanly into [numist/this-might-be-offensive/develop](https://github.com/numist/this-might-be-offensive) because this fork hasn't pulled 0bc8da696ceda1c8f9b1423058b232094cac22f5 and f9017257a4d21ab5dd75fedda130aa014374e36c.
Which Ruby and Rails versions are you using?. I'm not getting any error (Stack, Ruby 1.9.3, Rails 3.2.1) but  I'll take a look at it.
Yeah that is what I thought. You probably saw the issue I logged on the composer repo. Is there, or will there, be a way to avoid the fact that end-user need to add the custom repository to their package definition?
Agreed that this is fragile, but please ensure that Andy and Felipe know about this removal,
as they're working on GridField frontend behaviour and styling. They might rely on this "feature".
Hmm. Because we don't specify when getExposedVariables is called, is there a situation where you'd want an instance object? I don't like declaring functions inline - that seem's like API supporting a smell.

It does get a bit weird with TemplateIteratorProviders because we explicitly construct an instance to call the method on (so on TemplateGlobalProviders the method is called statically, on TemplateIteratorProviders it's called on an instance we construct)
next up navbar background color ;)
Fixed 3da503646aa7d54b0e26c2aa0a8b8e1b72071da9.
Added allArthropods as sheet 6. There's 374 total arthropods, and the 4 taxa of arthropods listed here adds up to 374 organisms.
And by "brackets" I meant "curly braces", sorry about this little confusion ;-)
@bgentry - awesome, thanks!
Yes, it's the root cause of the actual issue.
We'd better rename "#device" to "#selectedDevice"
A warning in Eclipse makes me realize it does work without in Maven 3. So I removed it.

You want to keep it ? New input for the CAS code conventions ?
Again, I'm not sure to understand why you don't use null as the default username attribute and get rid of the constant ? Less code...
I'd move down "no matter"
What is the reason for those new `from` methods? Java API? Should that be annotated in the docs, as we normally do.
this is not atomic with swapCell
underlying might be null here, someone else calls initialize and both are creating new UnstartedCell, potentially overwriting cell that has already been started to be used
Same comment here. Has it been tested on Windows?
um, shouldn't happen (internal api), but I have added some safety net, including watch of the termination hooks, please review
s/supervisorstrategy/SupervisorStrategy
Given that it does not make sense to add a deployment to `/user` himself I think the top-level element which you create is quite useless (or I’m stupid ;-) ).
Use explicit return type, and make it a one liner
Preference for ARFU or Unsafe?  Never used either, so will need to research
Fair point, but ScalaDoc is shoddy anyway. I'd like a documentation format
where the code samples is actually compiled and you could refer to the
different parts (perhaps you already can?)
$thisClass etc
I think that's reasonable.  One of the changes I've added to the provider module was to log where the credentials are coming from, so I went ahead and added log messages for keyring credentials as well.  I also added a log message when the keyring module can't be imported, but still let the ImportError propogate.

Thanks for the pull request.
All the three lines change semantic. That's OK if necessary or if it improves something, which I don't know. Otherwise I would suggest to only prepend `\r?` before the `\n' in the original version. Please comment.
Chef writes the stack trace in a separate log when it crashes. Although this stack trace is seldom useful.

We [should be able](http://wiki.opscode.com/display/chef/Chef+Configuration+Settings#ChefConfigurationSettings-logoutput) to set debugging to file and only info to STDOUT like this:

~~~ rb
log_level :debug
log_location "/var/log/chef.log"
verbose_logging false
~~~
Hmm. Can't recall why I did that.
So, when you tell it's alright, then these lines go directly to trash.
The first sentence here isn't complete.
The hot restart feature requires that the server sockets are not closed between restarts, so that the kernel can properly queue connections.
Use Structure.getField(StructField) here; it does the appropriate exception handling already.
In the other types, set_value is private and set is the main api. Maybe we could print a deprecation message here.
@stof no, this one is interface translations. It's custom and has nothing to do with Cucumber - Cucumber doesn't translate formatters output, only syntax.
`([^)]*)` this will match too much, try

```cofeescript
play = (d, s #) =>
```
what happens when editor.session is changed? i think it would be better to save only session or only editor
Just predictability -- in general I like to do as much as possible at object creation rather then spread across usage.
These should use the constants defined in TUIScroller.m, so any changes only have to be made to one location.
Well, this one is certainly true, but lets face the truth. Users that are exploiting such stuff when using core APIs, are already doomed :)
I don't know what happend. I thought I was only making changes to my fork of the repo, and this commit somehow got added to my pull request. Sorry.
Should this line be referring to <samp>fcm_make*</samp> or <samp>fcm_make2*</samp>?
Should have a rose suite-log-view link as above.
thanks. (on a related note, we should start coding up npairs)
Might want to do itemsize/sizeof(NPY_UNICODE). The compiler will probably implement it as a shift.
Eh, sorry, replied too fast. Your question doesn't parse. PyPi doesn't figure out versions. Do you mean pip?
This is wrong. This will try to read more data from a than allocated. Use emalloc+ 2xmemcpyorsuch ...
that is one reason of why zend extension does not support relative path.
I'm not crazy about this fix... It seems like the time when the issue comes up is when the user passes ``sample_names='*'`` - maybe we should handle that specific case by naming the file with ``_all_samples`` rather than the ``_`` separated list of sample ids. This fix also seems dangerous because we could potentially be overwriting a file in the case where the user generates two plots with >= seven samples each that have the same first six samples.
Is this supposed to be ```abspath(refseqs_fp)```? Also note that you do not need the backslash since this is inside of the parentheses.
Sorry but one more thing.
I think line of code should be highlighted by spaces by left?
You have to because it defaults to `element.last` so if it goes to `extract_values_from_collection` it will go to the last value which is incorrect. Like I said, the current default value for for `value_method` when you don't pass a value is `:last`
Sorry but this condition is wrong, the change inverts the logic. It should be something like:

```ruby
if include_seconds_or_options.is_a?(Hash)
  options = include_seconds_or_options
else
  # ... warn and set option
end
```

Please make sure you run the tests to make sure everything is ok. Thanks!
No more deprecation.
thanks, Jason
Eugene: when the desired end result is a collection of tuples, zip is fine
when you don't care about the intermediate datastructures, why bother creating them? unfortunately scalac does not deforest (which is what haskell compilers do: fuse operations so that fewer intermediate data/garbage is created)
I'd say that `isXXX` methods are preferable to `hasXXX`, because they are consistent with other `isXXX` methods.
I know it is ugly as hell. But it made a noticeable difference cases where
the resulting set is much smaller than the filtered set. Most pronounced in
the case filter(_=>false)  where the result is an empty set. Checking for
null in HashTrieSet.filter0 is quicker than calling x.isEmpty.

If this is too ugly for your taste I can take it out again and just live
with the performance penalty.

On Sat, Aug 25, 2012 at 12:37 AM, Josh Suereth <notifications@github.com>wrote:

> In src/library/scala/collection/immutable/HashSet.scala:
>
> > @@ -159,6 +169,16 @@ object HashSet extends ImmutableSetFactory[HashSet] {
> >            new HashSetCollision1(hash, ks1)
> >        } else this
> >
> > +    protected override def filter0(p: A => Boolean) : HashSet[A] = {
> > +      val ks1 = ks.filter(p)
> > +      if (ks1.isEmpty)
> > +        null
>
> Why don't you return HashSet.empty here and above instead of null?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scala/scala/pull/1163/files#r1458356>.
>
>
Tests are missing for atleast: andThen, transform (potentially more)
Ok, sounds good. Doing Python on C level is indeed messy (a similar situation with no C-level APIs occurs also with I/O on Python 3). It's probably best to move grabbing the pointer to a helper routine in a separate utilities header file. Also using the direct access hack could be considered if it is better encapsulated this way. But anyway, this is a minor issue, so it's best not to waste too much time on it.
I propose to write this as ``a Python slice object`` if it's even necessary to add the Python bit. Or ``slice object (a Python built-in)``.
Interesting fact. getMethods() is not a strict subset of getDeclaredMethods()...
  getMethods() returns public methods on listenerClass _AND_ its superclasses.
  getDeclaredMethods() returns methods declared by listenerClass.

We probably didn't realize that we were already getting methods from superclasses, but we were!
will `expand_mul` suffice here? or simplify's `._mexpand` (for multinomials and muls)?
Why are these indented more?
Yep, this is the spot.
This line is too long. Add a left paren before `to_cnf` and wrap before 80 chars, indent 4, and add a closing paren.
I reread PEP8 and see that I was wrong about the `*` -- I was running under the sympy-modifed version (that Aaron cites). So although we are doing exactly what PEP8 says not to do I think it looks better, too.
Whoops, will remove the envir argument. Agree - an options argument is simpler and more flexible than handling each argument individually. Will change accordingly.
in most cases we do not need `ifelse()`; it is less efficient than `if else` because

- in `ifelse(cond, x, y)` both `x` and `y` have to be evaluated
- in `if (cond) x else y`, only one of `x` and `y` is evaluated

in this case you do not need compute the second string if `fig.align=='default'`
Oh damn, here we go. I just realized that `CComponent` doesn't feature an `init()` method, so this will never be called. Teribly sorry.
i can do that
`$nillable = (boolean)$attr[3][$id];` No need to overcomplicate things.
I tried with 1.7.1 again and I get unpack errors.
Не пойму, здесь ````or```` используется намеренно? Почему вообще не сделать ````@lowest_point ||= ... ```` ?
不要引入path.basename
Ok, let's have an issue #3 - they are so cheap.
sure, it would be better to make separate cache_enabled init option, so you can pass another cache and turn it off for example
this will need "main" as well since it's not index.js
This doesn't actually prevent the rest of the task from executing. For example, if grunt was to log _after_ failing, it would still log. Which would be bad. For example, given this task, grunt will both log the FATAL error_and_ the subsequent log line.

```js
module.exports = function(grunt) {

  // Default task.
  grunt.registerTask('default', 'Default task.', function() {
    grunt.fail.fatal('This should stop grunt immediately.');
    grunt.log.writeln('This should not log.');
  });

};
```
`Adding` should maybe be `Accessing`
binmode should be applied before file read, and yes paperclip should implement this in his adapter
place.sports.map(&:name).join(', ')
What exactly is this doing?
unix epoc, not utc.
fix this as discussed in person


also test this build for this: I'm not sure that you can use :data: links this way without weirdness.
I think this needs to explain the operation a bit more clearly, or introduce it for more clarity.
I updated https://developer.mozilla.org/Project:en/Getting_started_with_Kuma#Syntax to reflect these changes. Check to make sure I understood correctly.
In terms of a test of three items, why not assert the entire order?

Can a node instance have a nil name?
Don't use this. It may not store properly (or store at all). Use the correct methods for it.
-----
Don't search for the textarea all the time. Store the wrapped node in a variable and use that variable instead.
Merged, thanks.
Please store these as actual image files in a fixture/ directory within this test directory and just read them from those files.
This should not global request. It should either take the $is_plupload as an argument, just like it takes the $upload_ary as an argument, or it should at least have $request as an additional (optional?) parameter
space before and after `=`
Just to work through this. This is ok to have as a default beause:
* This code runs with the current label, privilege, clearence etc' of the app.
* Presumeably, the collection label is likely lpub (any app should be able to query the database and insert rows).
* the policy has specified a document integrity related to who the document is "talking about". For example, in Gitstar the integrity of a row will be "gitstar" \/ [admin_username]. So a random app cannot insert a row with "alevy" as the admin since applyRawPolicyP labels the doc with the approriate integrity which will fail unless the app owns the right privilege.
* Similarly for saveRecord - the integrity of the existing row will not allow the app to modify the row unless the app presents the privilege "gitstar" or "alevy", in this case.
Ох. Что твой вариант, что мой не отличаются по производительности. В одном будет в одном месте чуть более сложный select, в другом - в другом месте. Но в твоем варианте ты нагружешь крон ещё одной, сторонней работотй, добавляя ещё логики
Это полная жесть. Мало того что это в Dao, так ещё и противеречит логике работы фреймворка.
These are shifted back to actual version numbers because they are brittle when addons inherit dspace-parent but have different version numbering schemes.
how this work? a single string is returned, what about repeatable metadata
&& orientation == Qt::Horizontal, looks weird in a QTableView otherwise ;)
Just in case - what is the license of the package?

We generally try to specify one of those listed in stdenv.lib.licenses (or add one when needed).
But where? There's just a document dumped on our local machines.
The word currently would ease a potential users pain.
Damn you copy & paste!! :angry: I missed a line.

Which reminds me, I was going to write a test to prove the caching is working! :wink:
Check that x is a matrix;

assert x.ndim == 2
where is the incref that goes with this?
To be clear, I didn't compile or specialize the deque class. This would have been a reasonable thing to try.
Needs a better name than "getLists" like "getPlanNodeList" to match "getScanNodeList"
65535 in range(0,65535)
False
Метод содержащий в названии Techs (мн. ч.) очевидно должен возвращать массив.
I don't see the usage of these private fields in the class.
almost useless doc comment
These lines are duplicates.
isWriter checks for function names of the format write* not set*
either the above comment or the function is incorrect
s/)/ /
It is the old note, which has not been hidden by github.

but you should have kept the empty line before the ``if``, just making it really empty
I agree. I'll see if this can be broken out into a utility.
I see. You could use an `alias` binding like this:

```
forward px = lambdaUnpack(closure);
alias x = px^;
var y = x + 1;
...
```

Now that @agemogolk has fixed aliases in lambdas, a nested capture should then capture `px` rather than `x`.
Same as above...

`jQuery.Tween = function( elem, options, prop, end, easing ) { ...`
Thanks for being so responsive to the suggestions!
There are two uses of `~` here instead of `!== -1` as well.  I know they were pre-existing, but we should probably fix these...
Yeah, it should be `standard`. Ignoring `newcap` for all globals kind of defeats the whole point of this option.
"Lernen Sie einige Tipps kennen ..." gefällt mir auch ganz gut.

hmm die Änderungen zum selben Branch von der auch dieser Pull-Request kommt pushen müsste Sie eigentlich hier anhängen, das ist dann im Normalfall auch so ok ;)

Wenn man allerdings beachtet, dass der letzte Commit hier 2 Jahre alt ist... naja wir werden sehen ob dieser Pull-Request jemals gemerged wird^^
I do not fully understand what does your method except opening url :smiley:
This can be replaced to Sys.openURL(URI) or need more checks?
Isn't this the exact same code as in core/library? Would it make sense to pull out a helper for converting backends to this map, or some central place to store the map instead of re-creating it many places?
Is this version tagged? Can we just get it via `:tag => 'v0.8.2'` or something?
ok sorry, i get mess up with the other part of the software.
Спасибо за внимательность :)
Пожалуй, «какому-либо» будет лучше. Поправлю.
It's not an array! It's a parameter list.
I would rather that indentation always be +4, rather than +2 in some situations. I think I started that after reading PBB.
Another case of TABs vs. spaces.
This makes sure that the config for this individual synclet is merged properly into the overall profile config object.
if (sprites...
no
if(sprites...
Would be helpful to show the default here, ``e.g. /Applications/Brackets.app``
The old code checked for "if (selectorInfo)" before dereferencing it. Are you really sure that it will always be defined?
putting an optional argument before a required argument is useless (and should be considered as wrong IMO) as you are forced to pass it anyway to be able to pass the following required argument
I'm not sure what you mean by "3x `!`". If it's `!!!something`, I don't know how this would help. Anyway, I pushed a simpler version, based on your suggestion. Thanks for that!
Just *This class defines document type for comments on wall posts.*
No dot at the end.
That took me way too much time to figure that out. Hope no one reads this.
In my installation, I had to change this to `File.join(page.source_dir` instead of `page.dir`. This was for paginating blog/index.html; as-is it kept generating the `page2` folder in the root directory instead of in `blog/`, as `page.dir` just returned `/`. I'm new to Jekyll so I'm not sure if this is a bug with the patch or a bug in my setup or configuration.
So to create a template now all you need to do is a have a container element with an id of butter-editor instead of an iframe?
This needs to be bumped over 2 spaces.
I didn't write this file, it's one that Atul wrote that I put in the tree. I mentioned this at the top of the file.
param please
So, _element.length is not a complete fix.

The only reason why the test fails, is not because the string is empty, it is because the string is not a valid element. That id does not exist in the dom, and nor should it. We should just cover cases where the string is not a valid id.

Should instead let that element in, and after that, check if element is null.

getElementById( "notAValidId" ) <- returns null, because that id does not exist.

So, checking for element.length only stops empty strings, but not invalid ids.

Something like:

element = document.getelementbyid( element );

if ( !element ) { return; }
Comma after "When specified", and apostrophe on "current editors". I also don't really understand this comment. Namely, the "...and will prevent it from closing if so" part. Can you re-word it?
Unfortunately not--totally reverse-engineered :(
API classes must only reference other API classes. You can't import anything from ljdp.minechem.common or ljdp.minechem.client.
Hmm warum sollten in der uri tags enthalten sein?
I like that this complexity is moved into the db_status method.  I also like that the db_status method doesn't set the flash itself, but leaves that to the code in the action.
I like to use the word 'should' in my then steps, so it's clear that the step is a verification both in the scenario and in the step def implementation.
This entire block can be stripped as we've handled the virtual case and the string manipulations further up.
You don't actually use this passed in variable so you can omit it. It's global anyway
Why are you chopping off the last character of buffer here? (It might be legitimate; I just don't understand.)
".., ignoring update"
Add a test or remove extradirs if not possible to write test
I'd probably throw some parens around this. Not a huge deal.
Ok, I"ve added flag %config(missingok) and checked it both against Fedora and RHEL. It seem ok.
Package rubygem-hashr is already present in Fedora. No need to put it here.
This include does not actually include anything, all methods in Ldap module are static so prefixing is necessary anyway. Does it makes sense to use mixin in this case?
This is either typo or I don't get this idiom :)
Meni ovo zgleda kao neuspjelo rješeni conflict. Nebi <<<HEAD trebao bit ovdje. Probaj ovo zbuildati
This code is duplicated because of https://bugzilla.mozilla.org/show_bug.cgi?id=731746.
should be wrapped in a small tag
That sounds like a good plan!
Well, there is http://pypi.python.org/pypi/importlib/1.0.1 ... we could put that `utils/compat` or `extern` if we really want to...
@lorenzo I am used to do this because our prof at uni always said:
"In comparisons use constants first"
This issue got fixed in [173524ad]
Yes. It's not clear to me yet what functionality you really want here. Throwing an exception will crash the simulation.
sniff. I ran php-cs-fix. Fixing!
Is `precision` really something that should be on a validator? This seems more like a post-processing step to me.
for the case of exactly one type, we should indeed use the rdf type that is specified - its even like that in my current code. i change that right now. seems to be missing some tests.

the exact line you comment seems right to me though. this is a mixed collection. we can't know the rdf type and need to figure out which one it is.
I did not know about this, excellent to know!
Do we want to keep this entry, but point it to the web instead?
not necessarily.  you can do this in many ways.  one is to make a ComputeServiceAdapter very similar to StubComputeServiceAdapter
Hm, even if there are no untracked or ignored files to stash, there could be actual changes in tracked files that will get stashed, right?
:sparkles:

I'd like some better namespacing for this. The `git_diff` prefix already collides with, well, with the diff code. We could rename all `git_diff_delta_` to `git_delta_`.
This is what I have right now. I have just borrowed strings that were already in KeeFox.

![Sample](https://github.com/downloads/dlech/KeeFox/sampleDialog.PNG)

I would like to change it so that it says either that KeePass is not running with a button to launch KeePass or that KeePass is locked with a button to unlock KeePass.

I would also like to have the KeeFox button down with the other buttons, but I have run into problems. 1. The OK and Cancel buttons do not have element ids. 2. It seems that the OK and Cancel buttons don't load until after prepareFill() is called so I can't get to them by other means.
Can you turn this into a negation? To prevent a indentation level

    if platform not in devices:
        continue
I'm not seeing this change, did you change this?
and again again.
Hm, but is it really necessary to have this test? Wouldn't we already get a failure above when calling utils.saveScreenshot() and bailing out?
yeah, this is a great idea!  I'm not sure what scope to store the cache in.  attach it to the page somehow, I guess?  I'm looking online how to do this now.
This is sad. We should file some serious bugs to fix this case-sensitivity.
This is actually causing an error that prevents the user from going back to the Add Release tab after switching to an other.
Signals? I think this handles that...
https://github.com/mozilla/zamboni/blob/master/apps/amo/models.py#L350
if the Z makes it work, I'd suggest updating the instructions directly instead of adding a note
This should either be measured in `em` or `px`. `ex` is only useful for sizing things relative to the height of lowercase characters. The JP version is mostly numbers, so `ex` doesn't make sense in this context. `0.5em` is probably more appropriate.
It is not used anymore, and the impact of removing it is considered harmless. I.e. paying customers are unlikely to be using or implementing it directly
i am using 2.3 for my tests i think there is no reason why locking it to 2.2 besides just being the version we were using at the time the tests got written
Why are `fi` and `sl` removed?
In a follow up change we should explicitly specify UTF-8 as the charset here. I DEMANDS IT.

Guava gets this right; they don't have a JSON constant; only a JSON_UTF8 constant.
http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/net/MediaType.html#JSON_UTF_8
@bschussek why `strlen((string) $constraint->min)` ? `$constraint->min` is an integer
Same for me. If everything went fine, let's the return be implicit.
But it is highly confusing for all people knowing Doctrine, as we knwo about the ObjectManager interface whereas we don't know about the EntityManagerInterface.
And btw, this is simply wrong. Your code does not accept an ObjectManager as the only method it calls on it is **not** part of the interface.
This is strange. This file must not be in your commit. Can you try pulling and pushing again?
baba fast reply :D
value的名字可能需要改下，容易让读代码的人误会。
Only boost::mt19937 compiles (not boost::random::mt19937) on my laptop (ubuntu 12.04, boost.random 1.46, clang). 
Like in the demo file: http://www.boost.org/doc/libs/1_50_0/libs/random/example/random_demo.cpp
Aside: I will take it on myself to take these type resolution routines and (a) settle on a methodology that avoids quite so much combinatorial explosion, as well as (b) try to find a way to make it generic so that this kind of code doesn't need to be fully repeated for every image operation.

I'll do this right after your "over" review is settled and merged, hopefully in time for you to use the revised version for the next op.
Again, will conflict with #119
Recommend a _SCUBA2_FTS_HELLO_ primitive that validates FTS is in play and jumps out the recipe if it isn't. You can either exit in good standing or exit and be upset about it. It's probably a good idea to exit with bad status so that the pipeline will exit with bad status when processing is complete. You normally have a problem if the FTS recipe is running without FTS in beam so something will need to be fixed (maybe the FTS is not in the beam but the FTS task thinks it is).
```js
var baseName = object? objectOrBaseName : nodeName,
    base = registry[baseName] || n[baseName];
```
I don't think group_id is defined here where this exception is being created.
You're right. This got changed between starting and finishing the documentation.
I think that here it should return `Undefined`
quickmusic etc. are profiles you've set up on the Denon beforehand?
This violates our logging statements
Do the .setDefaultValue(new ModelNode(false)) I mention in EJB3SubsystemRootResourceDefinition and enableStatisticsModel.isDefined() will always be true. That's more accurate as well, since if the operation doesn't define the value, that really means "use the default", not "don't do anything".
<br> убери, зачем он в <head> секции?
Once we resolve the issue of rendering, we need the gradient CSS code again.

I will recommend we keep these commented out code, but since we can find them in `git` record for now, I will merge this pull first.
I would use `this.stack.length = 0` but that's just me. I see a new array was allocated before.
remove this line, replace with the comment above.
umm where is the } at the end of the method? or am I wrong :)
I must confess it's a unsuitable comment... What I meant was the inits will start from here.
I will change this to a better description.
Maybe `.visible` is quite enough.
```css
.visible   {
  display: block;
}
```
It it normal that a specific version is used here? What about other versions of this tool?
updating it to `user_timestamp` since we're already using `user_id`.
Broader timestamping could be a good idea but I don't want to continue spending too much time on `vxpolls` as is.
right, this is a leftover from an experiment :-D will delete it
This, along with newSymbol, feels like it might be useful in more places than Matrix. Should it be pulled into RichPipe?
please change this to MatrixPipeExtensions because PipeExtensions is too generic of a name
can you change `[this repository](https://github.com/yeoman/yeoman)` to `[the yeoman/yeoman repository](https://github.com/yeoman/yeoman)`

otherwise lgtm!
Like before, validate replication.
Will open an issue if people run into issues and we see exceptions coming up.
Can you make the title shorter and use Camel Case in it?
missing newline (but i think that is resolved in the previous pr once that gets merged?)
Don't version OS/IDE specific files;