labofsahil/github-event-dataset-2012 huggingface dataset:
OSS4 is not defined on FreeBSD base system

so i think we use ifdef AE_FMT_S32NE and ifdef SNDCTL_CARDINFO
Easy now. If you to change it to something better please send me the changeset.
I sent a PR https://github.com/rails/rails/pull/5304
[src$="D.svg"]?

Matches only the end.
When I started thinking what is going to happen if there is an exception thrown in reciveAfterProduce I am leaning towards getting rid of sync optimisation. It will be confusing for the user as the exception thrown in sync mode might be caught by camel async processor. The same exception in async mode will be caught by akka dispatcher. This might be really hard issue to detect.
WDYT?
What is that for: `_G._VERSION` equal to `'Lua 5.1'`?
Oop.  Missing comma here.
@drogus that's fine with me!
here's an ephemeral note to go with the ephemeral patch
My bad. Should  be already fixed.
Slap a semi-colon on the end of this guy.
Can you please name the two `null` arguments using inline comments? For example:

```java
value = conn.invoke (objName, key, /* args = */ null, /* meta_data = */ null);
```
Все равно не компилируется:

```nemerle
using System;
using System.Text;
using System.Diagnostics;
using Nemerle.Extensions;

namespace A
{
  module M
  {
    Main() : void
    {
      def func(sender, e) {} // Option 1
      //def func = (_, x) => {}; // Option 2

      def process = Process() <-
      {
           StartInfo <-
              { 
           FileName = "FILENAME";
           UseShellExecute = false;
           StandardOutputEncoding = Encoding.UTF8;
           RedirectStandardOutput = true;
           Arguments = "ARGS"
      };
                    OutputDataReceived += func;
      };
    }
  }
}


```

та же ошибка
A complex number has False for both is_positive and is_negative.
what is the difference? I see less readability for the same functionality?
It would help if I could actually type the word storm though. Storm storm storm storm storm
as we are fixing the case problem, this shoudl remain as it was
CONFLICT (content): Merge conflict in test/unit/event.js
Thanks for inviting me to these reviews.
I'm not sure if this question is better suited here or the loongson commit, but do we need to handle different arch variants during serialization/snapshotting?
You're correct. Not expensive.  Just wanted to leave the statusline configuration in the user's hands.  I have no issues adding that back in.
Have you looked at the python style guide? It's here: http://google-styleguide.googlecode.com/svn/trunk/pyguide.html#Comments

This is not the correct format for describing parameters.
A) That's the exact reason I did this. It's annoying to have that code everywhere you just want a set of ids.
B) Fair point, though I think in 90+% of cases sets will have Batman objects.
C) `pluck` does sound better. @jamesmacaulay didn't like passing a string to `map` so we went with `mapToProperty`
I think it affects the fill colour of arrows, maybe?
нет,нет,Нет... бред как раз я уже вон описал как нужно строить ответ xml! если взялся как мы по xml то надо понимать как он работает
Во верно!
Google manages OS X desktops / laptops with puppet, why can't they use salt instead?
This won't quite work, but neither did my solution apparently. I did modify mine a bit and add some tests proving the issue I was trying to fix originally. Take a look at branch `js_with_compile_errors` https://github.com/edgecase/dieter/pull/3
+++ Michael Thompson [Mar 07 12 14:29 ]:
> I was consciously presupposing that the readers would always deliver `[..., Inline, Str ".", Str "foo", ...]` -- I guess for the sake of this kind of pattern matching, but I see this is fragile and hadn't thought of `--normalize`. (The problem I guess is `[..., Inline, Str ".foo", ...]` rather than `[..., Inline, Str "foo.", ...]` which would be okay for this bug.) In any case, is it just a question of a closer match:
> 
>      (LineBreak: Str ('.':chars): inlines) -> (as ++ [LineBreak], Str "." : Str chars : inlines)
> 
> assuming there is nothing suspicious about `Str ""`?  Or if so, the yet more verbose: 
> 
>      (LineBreak: Str ('.':[]): inlines) -> (as ++ [LineBreak], Str "." : inlines)
>      (LineBreak: Str ('.':chars): inlines) -> (as ++ [LineBreak], Str "." : Str chars : inlines)
> 
> If that's right, I can send a replacement -- or of course you can just revise it as you please! The bracketed `isSentenceEndInline (LineBreak)` and unspaced `++[LineBreak]` are ugly anyway as I see now.
> 
> The real worry I had about the patch is that it repairs a micro-bug I produced investigating the one reported, and I'm not sure there aren't others just like it.
> 
> By the way, is the man writer supposed to oriented only to reading with `man thisProgram` or also to allow `groff -Tps -mandoc -c`, which I was using to test since it gives the line where something is wrong. It seems a little touchier than the in-terminal reader in general, at least on OS X, and makes a mess of tables in particular, which I suppose are not that uncommon in man pages.

It should work with ps output also.

I haven't looked at the man output in some time, and you're right,
the tables are messed up.  Time to take a closer look.
Yeah. I screwed it up...
There's a way to do it cleanly... hang on...
Dann auch die Doku anpassen ;)
suchen und ersetzen fail... :D
у меня народ вот этой штукой пользовался пока спелл (http://ru.wowhead.com/spell=4073) не отключил
Механический дракончик http://wowfile.ru/forum/13-8-1
uiTextEmote, el codestyle de tc quitan el ui de delante ;)
Random other thoughts.... do we need to do anything with ES? i.e. don't send addon's to ES or remove from the ES index?
嗯  自动刷票功能  能使用了～
不过还有个小问题：

消息: 'classList' 为空或不是对象
行: 156
字符: 5
代码: 0
URI: https://dynamic.12306.cn/otsweb/order/querySingleAction.do?method=init
After upper, below message shows on doing ' git push '
To git@github.com:enkiKim/GitStudy.git
 ! [rejected]        master -> master (non-fast-forward)
error: failed to push some refs to 'git@github.com:enkiKim/GitStudy.git'
To prevent you from losing history, non-fast-forward updates were rejected
Merge the remote changes (e.g. 'git pull') before pushing again.  See the
'Note about fast-forwards' section of 'git push --help' for details.
*Thanks Recepemmi for report.
Correct -- I'd like to leave it out so that we don't have the dependency at the component level. As with the page controller, we can extend it to add a SignalManager dependency at the framework level.
Won't this fail if the authenticated user name is "stale"?  Maybe some other token than a String should be used, or maybe even a separate flag...
Apologies for the slightly confusing commit message - shouldn't use my shell history so aggressively I guess...
tu peux nous passer le listing de ta compilation stp ?
I should get points for keeping the total line count almost exactly the same.
Oh, the Android browser drives me nuts... I'll fix this.
Then you have to go change the database structure, and a whole host of other things.  With that in mind, what is the benefit that offsets all those changes?
:+1:

symfony1 is now working under PHP5.4 !!
How can this value be null on a file after the ViolationDecorator was run? Is it linked to the "best-value" for CoreMetrics.VIOLATIONS?
`INSTALLED_APPS +=`
The usage of XmlRootElement is now fine, as we have a jaxb-api dependency now.
See https://issues.jboss.org/browse/GUVNOR-1785
I wonder about the relative merits of this approach, vs letting the static middleware do the actual file sending.  so rather than res.sendfile()

    req.url = '/include_js/include.js'
    next();

The benefit of this approach is that I think you'll get proper `ETag` headers and the other headers the static middleware applies.  I don't know if the same is true for `res.sendfile()`
:-(  There's a screwup in there as now the
     file_put_contents($tmp, $parsed) 
line is gone entirely; it will return again later on though in commit 6e403647a62969f18a4967a335a56a81c2ef5e0e
Chris and I talked about this offline, and ended up figuring out that we both agreed upon the first interpretation. The tests to the contrary in this commit are incorrect, and in fact do not print warnings at all.

There is still a case for which this implementation doesn't follow the first interpretation, though:

```scss
.bar {c: d}
.baz {e: f}
.foo {@extend .bar.baz}
```

I believe the way to go about making a more correct implementation is to keep track of two additional data structures in the cssize visitor:

* A set containing all `SimpleSelector`s used in rules in the stylesheet.
* A hash mapping extended `SimpleSelector`s to the `ExtendNode`s that extend them.

Then, before `do_extend`, just iterate through the hash and print an error if any of the keys aren't in the set.
I think `msb_plus_one` would look better than `msb_plus`. Don't remember why I didn't do that in Python.
The Python code is the way it is because Python 3 removed support for `__cmp__`, which is the python equivalent of the spaceship operator. I wanted to have the 6.006 code forward-compatible with Python 3, just in case it ever gets accepted by the community.
Good catch again. Thanks for the quick fixes/feedback!
This is the power of rails routes. Take a closer look at the [rails routing guide](http://guides.rubyonrails.org/routing.html#creating-paths-and-urls-from-objects).
I wonder if you should check if the actor is still alive here, similar to how you do it on line 52 using the block form of #get. Otherwise, someone could manually #put a dead actor?
Also, theres an update.sh shell file which can
git pull, bundle install, db:migrate and db:seed.

Also, filters out a lot of the annoying bundle install output, except for actual changes to installed gems.
Tough call :) I would make it an error. There's no way it could
possibly be intended, and nobody pays attention to warning unless
they're errors.

On Fri, Mar 9, 2012 at 7:54 PM, slide
<reply@reply.github.com>
wrote:
> The question is, should I return an error, or just print a warning and skip parsing the file again?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/slide/ironlanguages/commit/5b0239ab0cb4277df6c53d88a24630be696266bc#commitcomment-1068450
Can you please undo the change on MinecartManiaCore:callEvent?
This change seems to make those of us without Nethrar installed on our servers throw constant error:

05:57:38 [SEVERE] Could not pass event org.bukkit.event.vehicle.VehicleUpdateEvent to MinecartManiaCore
org.bukkit.event.EventException
        at org.bukkit.plugin.java.JavaPluginLoader$103.execute(JavaPluginLoader.java:1026)
        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:57)
        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:453)
        at net.minecraft.server.EntityMinecart.y_(EntityMinecart.java:563)
        at net.minecraft.server.World.entityJoinedWorld(World.java:1256)
        at net.minecraft.server.WorldServer.entityJoinedWorld(WorldServer.java:110)
        at net.minecraft.server.World.playerJoinedWorld(World.java:1238)
        at net.minecraft.server.World.tickEntities(World.java:1145)
        at net.minecraft.server.MinecraftServer.w(MinecraftServer.java:527)
        at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:434)
        at net.minecraft.server.ThreadServerApplication.run(SourceFile:465)
Caused by: java.lang.reflect.InvocationTargetException
        at sun.reflect.GeneratedMethodAccessor3.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:597)
        at org.bukkit.plugin.java.JavaPluginLoader$103.execute(JavaPluginLoader.java:1024)
        ... 10 more
Caused by: java.lang.NoClassDefFoundError: org/akrieger/Nethrar/NethrarMinecartTeleportEvent
        at com.afforess.minecartmaniacore.MinecartManiaCore.callEvent(MinecartManiaCore.java:245)
        at com.afforess.minecartmaniacore.minecart.MinecartManiaMinecart.initialize(MinecartManiaMinecart.java:96)
        at com.afforess.minecartmaniacore.minecart.MinecartManiaMinecart.<init>(MinecartManiaMinecart.java:71)
        at com.afforess.minecartmaniacore.world.MinecartManiaWorld.getMinecartManiaMinecart(MinecartManiaWorld.java:91)
        at com.afforess.minecartmaniacore.MinecartManiaCoreListener.onVehicleUpdate(MinecartManiaCoreListener.java:223)
        ... 14 more

In the mean time, I'm restoring to an older build that doesn't require this...
commit agaoin
Need to use text() function instead of out() here and below.
you're right there probably is a problem here, will investigate...
Which emotes and how buggy?
The thing I don't like about this way is that you remove to ability to slow it down to be a step by step with a real low framerate.
I remember seeing that themeroller had some commits to parse around this line... wondering if this will require a change in themeroller as well (if it hasn't already been done)? I'm using the themeroller from a few days ago with 1.1 dev.
This is not exactly equivalent but should work in our case since we use that handler to give visual feedback.
I changed the whole thing
.. for an example
Heya
Compile error on linux ( debian 64 bit )

http://paste2.org/p/1890497
rofl, i forget it again
On Thu, 2 Feb 2012 12:47:22 -0800, Kirill Bychkov wrote:
> On my amd64 with 8Gb of RAM it dies with abort trap, but setting ulimit -d 60
> 00000 makes it playable. Also it increases speed of loading objects on start.
> P.S. This game has the craziest controls I've ever seen!

Oh, thanks. :)  I only have 4G of memory.  It should probably be
ONLY_FOR_ARCHS=amd64.

> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jasperla/openbsd-wip/commit/c056b531733f703f8fa9ef39886e28
> 9b53e4b90a#commitcomment-927780
> 
>
Haven't really thought about it yet to be honest. This actually works (vs. not working) so let's do this for now and come up with a better solution when needed :)
@pdrakeweb 

The  code here looks pretty straight forwards. Just a few things:

1. Could you open up a corresponding issue in redmine?

http://projects.puppetlabs.com/projects/commercial-puppet-modules

2. Also, if you have not already done so, can you sign the CLA:

https://projects.puppetlabs.com/contributor_licenses/sign

3. Lastly, rspec-puppet tests to verify the behavior change would be nice. I am happy to assist you with this process
crap! commit message went wrong. First line is missing
#63: Implement model unit feature

@eile: can you edit/modify while merging it?
You can also combine the apps onto lines - for example:

python manage.py test localtv admin submit_video

but django-nose would be nicer. :-)
Is $copay_update used as a boolean?  If so, make that more clear by making it TRUE or FALSE.
@kriswallsmith please enable Travis for the repo :)
語順を変えて「PHP の標準規約」はいかがでしょうか？
cool, just about there
Gemfile: `gem 'activeadmin', :git => 'https://github.com/gregbell/active_admin.git'`
Still isn't working for me Greg: https://gist.github.com/1699909
was ein zicken terror hier xD, Dreadii hat schon einen guten punkt mit dem private gehen. aber leider muss ich sagen ich binn noch jemand der OpenSource verfächtet und deswegen entwickle ich auch open. und OpenSource meint eben auch das jemand mit der Arbeit anderer Geldverdienen kann da habe ich auch kein prob mit. wo ich Dreadii recht geben muss ist wenn kein creadit gegeben wird, bzw im source die credits entfernt werden, wass genau genommen auch drastisch gegen die gnu/gpl verstößt. but thats life. und deswegen werde ich auch wieterhin, sobald ich wieder die lust verspüre, alles was ich bastle open stellen

und nu gehts los flamed mich ;-)
I think here we have the cause for why 2.0.5_rc1 started using much more requests. With this commented each refresh for an iteration triggers the events which in turn end up finding out updated metrics for each object in the iteration. @mike1072: what was the reason for commenting this code out?
This can't be correct, exit() the method no matter what??
I have modified the complete getRootPageFromUrl() method, so the diff view looks a bit ugly.
i run this sql files, it was a core problem, u fixed
Makes sense. I updated the pull request.
The complete list of stuff that could come up is in the conf/checkstyle.xml, but it might be useful to see the complete list of checkstyle comments that have been trigged. You would have to search the db for that, filter by Coment.type ='S' (static).
Looks great! =)
No no, es una funcion de ICC solo, el segundo .cpp es el tiempo de despawn del moco tocho al explotar y el 3º es para que hender armadura se stackee entre los wars, lo explique en el issue del R2
this arouses memories.
Retails vmaps are less strict than TCs some things on TC have LOS block while on retail no.
You can obtain 130 rep/game = 13 ticks.
#define BG_AB_ABBGWeekendReputationTicks    150 - wrong
1600/150 = 10 - only 10 ticks

#define BG_AB_ABBGWeekendReputationTicks    120 - right
1600/120 = 13
awesome! i was following your progress on the dedicated branch, good job, anyway making some adjusts on db and it will be a complete one, ty
Hmm. Nu tror jag du missförstod mig. De parenteserna gör ingen nytta......
There is a discussion here about rewarding you for destroying enemy rail (which feels more intuitive to me): #375

If you want to suggest the opposite (punishing you for destroying enemy rail) feel free to start a new issue for it :)
Sent you my levels :)
Thanks for the handcrafting merge! :-)
1,2ghz its good for battery yes. We don't want to cap the device while screen off as it hurts race-to-idle.

The mounts are good enough, I never had a problem with it. The only one that could be problematic is barrier=0.

On Jan 16, 2012, at 5:45 AM, Renaud Allard wrote:

> write /sys/devices/system/cpu/cpu1/cpufreq/screen_off_max_freq 1200000 is probably not a good idea battery wise and there are no reasons why the cpus should be allowed to go that high with screen off. It could even be a good idea to put 350000.
> 
> the mounts are somewhat not secure enough
> 
> the net buffer sizes won't improve anything in most of the cases
> 
> the idea is to not do things that would degrade experience by losing data or consuming more battery than stock
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/renaudallard/gnexus_kernel/commit/2a556388739ca755944bde3257b8c98abd6e3f54#commitcomment-869783
Commit erratum : "Clone: adridu59_hallon_defconfig -> adridu59.miui_hallon_defconfig"
Maybe revert this...
This is still in 1.1.1 not in jezdez/django_compressor which is a pity. Will try influence this.
oh damn even typo in message -.-
Thanks Venugh to pointing out
Ahh... I didn't think of doing it like this. Nice.
Works fine for me.., this commit along with setting it in build prop
Sorry, where is the repository? Didn't it automatically created?
Hi Aleksi, is the formatting changes like indentation, tab and newline? I'll try to not changing them as much so its easy to see the changes on here. However, I believe github sometimes show a single change as a whole bunch of deletion.
Right I was thinking different type of `Node` (and probably `Path`. But I am not sure this can be done without requiring subclassing. This needs to be experimented.
What I mean is that a constraint at a getter method can be interpreted as property-level bean constraint but also as method-level return value constraint. So it would be validated when calling `Validator#validate()` for the bean, but also when performing method-level validation when the getter is invoked by some code and the bean is enabled for method validation.

Now the question is whether this behavior is ok or not. In HV it's currently like that and so far I haven't seen any problems with that. But I think in OVal one can explicitly configure whether a constraint at a getter should be interpreted as invariant (property-level constraint), post-condition (return value constraint) or both. So maybe we should have something like this as well.
Why we need to vendor gdata gem ?
If gdata has a bug it's better to fix gem and add increase gdata version in dependencies.
just remove and then add a .gitignore file with *~ in it.
A étudier, je ne voulais pas que l'utilisateur retourne forcément sur la page teaser.
Sono abbastanza affezionato al max 80 linee per carattere. Pylint docet
Arbitrary. I chose Y for lack of a better option, since I prefer not to execute from the global context.  But maybe that's fine?
If a file name contains "#" or "!" or "@", there will be problem showing these files. A simpler way might be just looking for "http(s)://" to decide if it's relative or absolute URL.
Import within a function is illegal (see http://docs.python.org/2/howto/doanddont.html#inside-function-definitions). Please change this.
Actually I can imagine situation that we have widget on page with page owner A (group page or sth?) and widget is being created by one of several users with permissions, let's say B and C. Both widgets are connected to A by private setting despite being owned by B and C respectively. It seems to make sense after all. The word from person who designed it would be very appreciated though.
Resume and TryResume?
When I read "Acknowledgements" I expected a list of names.
Perhaps something like "Delivery acknowledgements"?
No, but for the casual reader or the guy who works with this test, it is nocer to know instantly if its a count or a duration.
This is a FIXME, please revisit all TODOs here and make sure that the ones that have to be done are FIXMEs
What was the reason we couldn't use Typesafe Config?
and since it's only used once, it's not needed :-)
too long line, doesn't fit pdf
it wouldn’t if it were properly bounded against the deadline (as you have now done); since this is for tests, exponential back-off versus straight polling does not seem to be a hot topic
yeah, original author seems to like .empty ;-)
I'll change
just to add more insult about the meaning of "interleaved": some scanners (e.g. Philips we have)  have different idea behind what they call "interleaved".  On our scanner interleaved corresponds to slices going at sqrt(number of slices) distance and "default" goes with conventional odd/even.
Nice catch on these internationalization snafus
all these breaks and continues are definitely not making it more readable.
Why filter out stuff that already has a thumb?  The thumbloader thread shouldn't touch such items.
For niceness, you might want to have EC2.init_reserve_costs() take an init-value:

current_simulation_costs = self.EC2.init_reserve_costs(float('inf'))

EH?!
!shouldBool
Пётр, что-то я туплю. В чём смысл этого обработчика? Почему то же самое нельзя с клиента сделать?
`parentId` вроде есть на клиенте, дальше GET /_/case/xxx.
please make test name more descriptive, i.e. 'should be updateable if the MiPlan has no MiAttempts'
Good idea: ```echo "public/css\npublic/js" >> .gitignore```.

I'm working on the deploy process now.
wow. so simple! :)
2012/6/18 Robert Carlsen <
reply@reply.github.com
>

> > +#include <sstream>
> > +#include <math.h>
> > +
> > +#include <boost/shared_ptr.hpp>
> > +
> > +/// a smart pointer to the abstract class so we can pass it around
> > +class Function;
> > +typedef boost::shared_ptr<Function> FunctionPtr;
> > +
> > +/// abstract base class
> > +class Function {
> > + public:
> > +  /// virtual destructor for an abstract base class
> > +  virtual ~Function() {};
> > +  /// base class must define how to calculate demand
> > +  virtual double value(int x) = 0;
>
> Might we overload the `()` operator to so that to use a 'function' we
> could simply:
>
good suggestion, added it in the most recent commit

>
> ```
> myfunc = LinearFunction(s, i);
> val = myfunc(x);
> ```
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cyclus/core/pull/197/files#r1004033
>
duh!  I guess I was so focused on the doc build that I didn't think to test the normal build.
Because you might not want to go and iterate on all the loaders you have in order to ask them something that could be cached anyway. If you think it will bump the resource usage in terms of memory too much I'll remove it but else I don't see why we should remove it.
Accidental newline?
Il faudrait localiser ces strings là une fois que ma branche de localisation sera mergé.
je sais pas.... cest vrmnt biz
Nope.. Let me fix this.
Chris,

Thanks for looking into this, but you statements are not always true.

Technically, any transform type CAN be placed in '.txt', '.mat', or '.hdf5' or '.h5'.  These are all part of the polymorphic transform read/write mechanism of ITK.

Warp fields ARE COMMONLY placed in .h5 files (we do that all the time), and so are Affine transforms.  In our lab we are converting to only using .h5 format files for all transform types (translation, rigid, affine, warp, etc…), and we have a long history previously using only '.mat' format.  We have avoided the .txt format for many years due to the inability of ASCII format to accurately represent floating point numbers.

Finally, the file name extension in only a lab convention and fixed to the type it holds.  There are other less used transform file writers available (i.e. '.hdf4') that may be used.

Hans



=================================================================
Hans J. Johnson, Ph.D.
Assistant Professor, Department of Psychiatry

Mailing Address:
W274 GH                             Email: hans-johnson@uiowa.edu<mailto:hans-johnson@uiowa.edu>
200 Hawkins Drive                   Phone: (319) 353 8587
The University of Iowa
Iowa City, IA 52242

From: Chris Filo Gorgolewski <notifications@github.com<mailto:notifications@github.com>>
Reply-To: nipy/nipype <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Saturday, October 20, 2012 4:55 AM
To: nipy/nipype <nipype@noreply.github.com<mailto:nipype@noreply.github.com>>
Cc: Hans Johnson <hans.j.johnson@gmail.com<mailto:hans.j.johnson@gmail.com>>
Subject: Re: [nipype] ANTS: Removed unnecessary copyfiles and added output_image option. (#471)


In nipype/interfaces/ants/resampling.py:

>          if opt == 'transformation_series':
>              series = []
>              affine_counter = 0
>              for transformation in val:
> -                if 'Affine' in transformation and \
> +                if transformation.lower().endswith("txt") and \


I'm glad that I've asked. But basically there are only two possible types of transforms - affine or a warp field? An a warp field should never have an "txt" , "ma", or "h5" extension?

—
Reply to this email directly or view it on GitHub<https://github.com/nipy/nipype/pull/471/files#r1896910>.



________________________________
Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged.  If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited.  Please reply to the sender that you have received the message in error, then delete it.  Thank you.
________________________________
No yours, but '{' should be on next line.
There are some slightly unusual ways of importing that do not do this, usually by tools that are hooking into the import mechanism. Yaroslav found that ropemacs is one such tool. I have encountered it several times with nosetests depending on the directory where I start it.
What happens if `putInCache` fails? Failure needs to be logged as well there must be code in place to continue the request without failure.
What happens if the key is longer than what the cache can take?
You've called `create`, I believe you could just check for `persisted?` here, to avoid running validations again.
You could probably cache the `convert_key(key)` method.
good catch. updated.
I just commented out the `elsif` branch and ran the Action Pack test suite - no errors.
You can use string formatting:

```
    url += '&s=%s' % size
```

Not sure which I like better, though.
Of course. Sorry for the noise.
Odd, why not `def message(kind: String) = ...`?
If you are Emacs users, this thing is a timesaver: http://www.ymer.org/amir/2010/01/28/pyflakes-flymake-for-python-programming-in-emacs/
"A Voronoi"
Sorry, defined above where?  This is the only ``expm()`` in this file.  Or do you mean the one in ``scipy.linalg.matfuncs`` as per your other comment?
Nit-pick: indentation here is one character too far.
It's still available -- the code is just hidden away so it doesn't clutter the main namespace.
As with subs, I think it should be possible to do it both ways, depending on what you are after.
Be careful using mutable containers as default arguments.
you can probably figure out how to do the same thing here as above, using the jon function.
This is not needed for the lines that were added here.
Concerning docstring style: when there is a problem the Sphinx doc
builder will raise a warning. Thus, what I do is to write the
docstring however I like (following only the obvious rules) and only
at the end of a project I add the documentation and fix all the
warnings in one pull request. This way you won't have unrelated
cosmetic changes in commits related to algorithms. It makes everything
more readable.
Why so much whitespace?
Go ahead and enter the matrix that is the answer like you did above:

```
assert (F ==
Matrix((
(0,-B_x,-B_y,-B_z),
(B_x,0,E_z,-E_y),
(B_y,-E_z,0,E_x),
(B_z,E_y,-E_x,0)
))
```
That will hopefully change (see issue 2865). IMHO this should work the same way. You can always use pprint to get something that looks nice. 

At any rate, the way this looks now is clearly not very good, with only half in 2D.
Why changing from a list to collection?
Good catch. But Looks like I can remove this because build worked when I caused it to download all of its dependencies.
Set up _your_ shell login script to call the opencmiss_programmer script, which will set environment variables used when building OpenCMISS.
https://www.pivotaltracker.com/story/show/36845125
I guess you can drop this file.  On Windows, we only support MSVC.
Need to uncomment this code to show '?' when needed.
But I don't know how to get the team
Probably should take this out.
can you confirm?
Actually this was correct. Only one group is retrieved.
In this case a comment should be added detailing which function requires this. And consider rewriting said function to not use the global.
Do sql_in_set once and store resulting sql in a variable?
Use `$post_id`. `$id` is meaningless.
Okay then it is fine.
I think another constant should be added and MAX_FILE_AGE should not be used. These two values are independent in general. One constant defines how old the files have to be in order to get deleted. Another constant should define how often the cronjob is run.
true/false instead of 1/0?
Nu mai e necesară acest `if action == 'value'`, se poate face filtru direct pe SecurityConfig.objects. Dacă nu e niciuna cunoscută, nu face nimic.
Hmm... `Procfile` looks like a Heroku-specific thing.  (Unfortunate that it's not called `Heroku` instead (similar to Travis's `travis.yaml`).)  Some ideas to address this confusion:
* Add a comment to this file, which explains what it's for.
* Remove the file entirely, and update our docs to explain how to get a mojito app running on Heroku.
* Better yet, I think Heroku with function just fine *without* a Procfile if we update the app's `package.json` such that `node start` starts up mojito.
"the  given" has two spaces in it
Didn't we have a discussion that `var boundModel` was an error? I think @jchannon likes to tell people that I made him introduce the bug ;) Also https://github.com/NancyFx/Nancy/commit/572fb4f64c70a94dbd2fceb94a0fc24c7202cfa8#commitcomment-1875833
wouldn't it make more sense to separate contrasts out?  i.e. now it is difficult in 1 call to fit the model and get all nice aic, bic while also getting a contrast stats out...  also it is impossible to have multiple contrasts right?

I just wondered if it would make to may be to add argument 'con'? and then add 'con%d' sa's per each contrast?  or may be even if contrasts specified as dictionary then those keys could be used to name corresponding sa's which would carry all those 'tvalue'/'fvalue' attributes. e.g. if I specify dict(faces=[1, -1], houses=[-1, 1]) I would obtain .sa.faces and .sa.houses with corresponding stats. (.descr still would be there describing the 'fit').

and if I specify res='faces.tvalue'. corresponding t-values would be placed into .samples thus allowing easy reuse e.g. for feature selection

just a thought... now I just got confused by multifunctionality of 'res'  I guess

Then 'res="
Fine for now....for some reason we have 2 geolocation files in library.
nit: deleteDiagnosed and referPatient are camelCase, while diagnosislist and diagnosisfilterbysearchfield aren't.

Probably should change these all to camelCase, later.
In the bluegene plugin select_g_node_init could alter the number of cores available per node.  I like the idea of moving this code to common though.  Perhaps instead of calling the static function _init_global_core_data in read_config.c it could be made an extern function init_global_core_data (or renamed something else) in node_conf.c.  That way each plugin could call it when it were ready.
Comment confuses me. It appears that the ```try-except``` will mean it continues to load even if it hits an EOF in one of the files.
Remove "public" from methods description
This line is very long. Can you break it ?
I'm planning on fixing this in the parser refactor so I can go through and change all the node tests to the updated code then. Hopefully that won't entail much of a change in the unit tests.
For these comments, can you do them on two lines, so they look like this:

```js
// Support: IE >=9
// Fails to persist the checked state of a cloned checkbox or radio button.
```


Same thing for the others with similar browser specific notations. Thanks!!
Use MADLIB_WRAP_PG_FUNC(), as this function may cause ERROR.
Perfect!

On Wed, Aug 8, 2012 at 7:58 PM, Mike Bannister <notifications@github.com>wrote:

> In packages/oauth1/oauth1.js:
>
> > @@ -0,0 +1,114 @@
> > +var crypto = __meteor_bootstrap__.require("crypto");
>
> @kennethkoontz <https://github.com/kennethkoontz> Meteor's http package
> can be used on client and server. http://docs.meteor.com/#meteor_http
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/meteor/meteor/pull/251/files#r1338946>.
>
>
Oh, if you want to do some testing yourself maybe try throwing this at the top of Skill.Run():

if (!(User is Player) && EvalTag(PowerKeys.NeverUpdatesFacing) > 0)
    User.TranslateFacing(TargetPosition);
I think you missed turning this one into an elif. (Totally understandable--there were a few.)
Need to pass `prefix` to the build system somehow, usually:

```rb
system "make", "PREFIX=#{prefix}", "install"
```
Okay I'll seem if I can get the author to create versioned downloads.
Maybe add `tool = tool.to_s` here so that it can handle symbols as well.
ok, got that with

    def install
         system "python setup.py build"
         system "python setup.py install", "--prefix=#{prefix}"
    end

I will try to find a solution to your other comment first.
It reads better if the condition for success is checked first and else we return nil instead of the other way around.
Hey Jack, what do you mean by that? libxml2 is a runtime dep of pathfinder.
OCP\Config::setAppValue would be better here
We could, what we can't do is add the code to handle the "import from itunes" checkbox.  I thought it would be strange if we handled one checkbox inside MusicSetupVBox and one checkbox outside of it.

On the other hand, it would reduce code duplication.  I'll change it.
Nope my bad.  Why are all the headers in their own folder anyway?  Will remove...

On Oct 6, 2012, at 9:48 AM, Dmitriy Kalinin <notifications@github.com> wrote:

> In Source/Headers/CDRSlowTestReporter.h:
> 
> > @@ -0,0 +1,5 @@
> > +#import "CDRColorizedReporter.h"
> > +
> > +@interface CDRSlowTestReporter : CDRDefaultReporter
> > +
> > +@end
> do we need this file? if not it's also inside xcode.proj.
> 
> —
> Reply to this email directly or view it on GitHub.
>
This should reference the setting name in `praw.ini`. I believe it's called `ratelimit`.
I'll rephrase the last line to


   Provide either the subreddit id (sr_id) or the subreddit name (sr_name).
Looks like you forgot to finish that thought  :)
@rmocanu use spaces instead of tabs
nen schalter wäre nett, wo man einstellen kann ob die CS probleme nur "erkannt"  oder auch gleich gefixed werden sollen...
Addressed in: 68793d2
The calling code just casts it back to `Transport`. If you have better ideas I'm eager.
Hmmm, we should probably normalize the path first somehow. I'm not sure if there's a method in the `Path` class to do that. Since we're only allowing selecting a directory, it should be safe to assume the path is a directory path. In that case, I think having a method `EnsureTrailingSlash` which appends a trailing slash if there isn't one would be great.
This actually kinda confused me since the naming scheme is similar to the name of the function calling this, but this inner function here does something very different and always returns a new record. Can you make it like `newProxyForKey` or just leave it or something?
Yes it is.  

The command is debugBreak().  It even says "block to break".  If he also wants to test that drops are spawned, then that is separate, he still needs to clear the target block.

We also need a method which will return the face that is hit.
These don't look right. You're setting the icon to use the load screens?
Equal sign alignment is weird on this line. Seems like you should either remove extra whitespace, or align it with following var statement.
this is meant for older browsers - delaying the need for `localStorage`
But true, it should be `!window.localStorage`
Should I make it async ?
It is not necessary to set localStorage at every revision save. You could only do this on every exit and every minute or so.
Si estoy en el doPost significa que el chabon pudo hacer post, con lo cual tuvo que haber pasado por el get, por lo cual tiene que haber tenido permisos. O algo esta fallando en mi razonamiento?
Also I think you should be able to use req.couch_user.get_id (which is an alias for the doc's _id property)
turn that 8.0.3.xxxx into a property (dep.jetty.version) so that any update to that version affects all the jars.
This applies to most of the methods in here where obj is used as a variable name.
Ok. I'm going to implement this... Didn't realize `git_repository_odb` and the likes where allocating memory. I'll have a closer look at the libgit2 APIs.
Yes, `middleware.py`. I would put it into `account` application.
Don't bother with the ./ prefix, it's implied.
The zoom controls on the map take care of zoom settings.
...and we use camelCase for variable names. https://github.com/mozilla/pdf.js/wiki/Style-Guide
Return early to avoid long indentation below:

```
if (!pem)
  return;
```
Why did you remove this alias?
I like this. It's much cleaner
At least on my system, this line is not required. Using
DateTime.parse(date)
works just fine.
This doesn't have to block this patch for now, but I'd really like to have a set of clean path utilities for e.g. creating a relative path from an absolute path and base path, rather than using string manipulation. This might be a use case for Boost, although I guess there are MS APIs that do this as well.
I do not recall but yes we can do that.
加一个tag，区分模块。
内存分配不了不应该返回NGX_AGAIN，因为这时如果重新分配仍然会失败，应该返回NGX_DECLINED。那么在ngx_http_read_non_buffered_client_request_body中，rb->buffered无用，所以rb->buffered可以去掉。
在后面rest==rb->rest的时候，应该根据rc为NGX_DECLINED或者是NGX_AGAIN分别进行发送或接收的操作
Potřebuje plurál, jinak použít tuhle neutrální formu i ve zdroji.
this particular constraint requires the uniqueness of the combination of the keyname and value.

an object can have custom info like this:
1. add keyname: "user" and value: "lzap"
2. add keyname: "user" and value: "msuchy"
both validate successfully

but this will fail:
1. add keyname: "user" and value: "lzap"  ==> successful
1. add keyname: "user" and value: "lzap"  ==> validation error
No, that's a good point. Thanks for pointing that out.
For now, I'm going to leave this one in the content_view_version model.  I need the request to be scoped to the version's list of repositories; therefore, to move it to the Repository looks like may have to move the entire query; otherwise, scoping will get lost by the with_exclusive_scope (e.g. joins..etc).  If we need it elsewhere in the future, we can re-visit the implementation.
Put the opening bracket on next line.
When we are creating new table, can you please create indexes as well. I know that we do not have it in past. But for new tables we should do that.
I would suggest here at least:
UNIQUE INDEX(informable_type, informable_id)
Any particular reason these fields are all declared as static?  Sorta defeats the point of using C++
I should hope not! Changed to `!charset`
Why rely on exceptions?
Maybe show the result here, rather than just setting it to a variable.
Like in convolve_fft, you can't use "Options" - instead, I'd suggest e.g.,

```
kernelwidth : float, optional
    Width of kernel in pixels  (see definitions under `kerneltype`)
```
Typo: `guarentee` -> `guarantee`
You can always make the various enum related method protected as well.
Why not add _remoteOptions to the OptionSet? Why not use _remoteOptions (if present), and fall back to defaultRemote's credentials? git-tfs should work equally well for someone who stores creds in the remote config once, and wants them used everywhere, and for someone who always passes them in on the command line.
Are you saying that there are different syntax highlighting modes for the magics and their names begin with "magic_"?  Can you provide a bit more background/overview about how you are doing this?
revert new line
just a note, but I'm inclined to implement this as Optional<URI> at some point with jesse's hints on gson binding.  no action needed.
`git_object__dup`? I hate to be poking into struct internals in code outside of `object.c`.
You are correct that this logic is no longer right, but I think the first call to `print_cb` should be made even if `delta->binary` is true.  I think this should read:

```
	int result;

	...

	result = pi->print_cb(pi->cb_data, GIT_DIFF_LINE_FILE_HDR, pi->buf->ptr);
	if (result < 0)
		return result;

	if (delta->binary != 1)
		return 0;
```
Why is there a empty option here?
These are puzzling. None of these are localized as near as I can tell. Is that a bug?
`<meta charset="utf-8">` is enough in HTML5
This isn't a really helpful debug log statement. Can you prefix is with something like `logger.debug("Extension: %r", args[0])`?
I find that name confusing regarding that we also have a ``services_list``. Also, ``service_list`` doesn't say much about what it actually is. (Why not call it ``implementations_list``? )
Yup, but letting it in prevents it from working on my gh-pages, so I had to decommit it to test :(.
I suggest you readd it, I don't see another way to go, I'd be glad to do it if you know one


--  

Romain Dardour, hull
romain@hull.io (mailto:romain@hull.io)
+33 (0)6 20 555 666
19, rue de Milan — 75009 Paris
Please note : Oahu is now hull !


On Sunday, October 28, 2012 at 6:17 PM, Mike Perham wrote:

> In CNAME:
> > @@ -1 +0,0 @@ > -sidekiq.org (http://-sidekiq.org)  
>  
> I don't think you want to delete this from my repo, or else sidekiq.org (http://sidekiq.org) will stop working.
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/mperham/sidekiq/pull/472/files#r1961521).
Implicitly passing the ENV superglobal is a bad idea imo, as it could contain sensitive information.
why did you decide to leave some out?
shoudln't it be MinCount and MaxCount ?
I think you should remove these PHPUnit annotations
I did not knew you require a blank line before a return.

strange because 10 lines about, in the getTableMap method, there is no space between the return statement and the comment line.
it does not need to be public, right ?
if `$default` has been sent into the method you are converting it twice instead of returning it right away, same if it is null you should just send new \DateTime
Simple `if`s would be a bit more readable IMO =)
Not sure if it would be redundant, but a TODO indicating that the scheduler.put() that execute() runs might be blocking could be good.
It seems like ordering matters here, unless I'm misunderstanding the table structure... what happens when a forwarding is updated and then later deleted, vs when it happens the other way around? Or will there be exactly one row per forwarding?
The +1 icon and the reply icons both bounce 3 times.. total of 6 animations each.  But the reply icon starts bouncing after the 2nd bounce of the +1 icon... in a sort 1..2.. fashion.  So it is correct that the +1 animation chain does not set self.currentlyAnimating = NO, as the reply icon is still animating. It is also the reason it was a little difficult to DRY this up. I suppose with something like:
```objective-c
-  (void)dribbleAnimationWithView:(UIView *)view delay:(NSTimeInterval)delay secondBounceCompletion:(void (^)(BOOL finished))secondBounceCompletion thirdBounceCompletion:(void (^)(BOOL finished))thirdBounceCompletion
```
would DRY it up.. but the nested dribbleAnimationWithView call in the second bounce completion may not be all that readable.
well, this chaining of ifs is not much readable. I think that it can be changed to more explicit "state machine"
```
if (action == "disable") {
  if (is_enabled)
    return serviceDisable (name, false);
  else
    return true;
}
else if ...
Why here you are casting "this" to String?
Does a trailing slash break things?
I think so, its the same as a missing texture file. You wouldn't want to return "density" if the user asked for "temperature" for example.
This example file is a mess. Rather than start fiddling/cleaning it'd be better to leave it untouched by this PR.
@magnify What is your take on this?
 http://ding2tal.ting.dk/profiles/ding2/themes/latto/tools/tdcss/#miscellaneous specifies that commas should be in the HTML.
Commas are easy to add with css if you want them, but hard to remove with css if you don't want them.
So I think the commas should be removed from the HTML.
why not use dj_database_url here? This looks just insane, no offense. At the very least use urlparse properly here.
Ah, ok, might worth adding a comment so someone else doesn't try to optimize like I would have :)
Bring this line back.
PEP8 Nazi
Part of that is implemented here: https://github.com/iFixit/wompt.com/blob/master/nodejs/public/js/ui.messages.js#L43
Happy to use hasOwnProperty instead, but if we do it on the cache object should we also do it when testing properties on the view? It would mean any legitimate use of the prototype change would stop working. Eg:

```javascript
var view = new UserView();
view.foo = 'bar';
view.log( context.foo ); // 'bar'
view.log( context.user ); // 'jaffathecake'
```

Using hasOwnProperty makes view.user invisible to templates.
@mikesherov We should probably be using `.simulate()` for all events, but in practice we generally use it as needed. At some point I'd like to finally improve simulate, maybe I'll make that a priority for the 1.11 release. We can go down the road of updating all tests to only use `.simulate()` and then start enforcing it.
That is my concern - a child process not dying in a clean way.  What can we do to guard against data corruption, etc.?
OK, I get it now.... :P

Can you guys add a comment on Line 189 about this or move this line to L191? Just to easy the future possibility of confusion. Thanks.
We should rename `displayThread` `fillNumber`
@jmcanterafonseca make it sense to try to use same firends_list.js/contacts_list.js instead of duplicating?

cause we could have bugs duplicated and resolve them in one plance and not in another.
+1 I totally agree
I will remove it for now and just leave AccountCreation so we can do all those changes at once I guess.
`this.alarmList = alarmDataList`