labofsahil/github-event-dataset-2012 huggingface dataset:
If we needed to check for multiple prerequisites you could do something crazy like this:

```json
"prerequisites": [
    {
        "type": "prerequisites",
        "value": [
            {
                "type": "skill",
                "value": "magic"
            },
            {
                "type": "treasure",
                "value": "flying carpet"
            }
        ]
    }
]
```
I create a cookbook to manage htpasswd. What append if apache isn't installed (for example)?

http://community.opscode.com/cookbooks/htpasswd
No it's not.The first locator refers to : "This theme is incompatible with your version of Firefox", and the second one refers to the incompatibility message from under each compatible theme, after the hover.
this is not needed, plz remove it
the define after the endif should also be commented: #endif /* __x86_64__*/
This can be done as a list comprehension:
```python
json.dumps([{'x': x, 'y': y} for y, x in results])
```
Thanks Kannan, I have removed the disabled lines which have commented out while testing the app.
All 4 of these CREATE statements should use single quotes.
I said it was a reflex, didn't I? Will change it, thanks for pointing that out ;-)
I can eve, push them myself ;) I'll do it this evening.
Because just setting "done" to 0 on an entry with an all-zero mask means the entry will not get back to done == 1 before the next version change and subsequent mask reset in eit_status_find(). In practice, this means the scan will not be done until the now/next table expires.

If you intend to keep the EIT status over epggrab_ota_begin() invocations, would it not be better to remove the entire "Begin" snippet from _eit_callback() and set dont to 0 in the "Reset" part of eit_status_find() ?
yep. thanks !
small bad mkay. ;-)
temporary cache of the functions to be added at execute time
I think the view should have some logic in it that says 'if :mi_plan errors, then render an error' since the view is in charge of how things look.  The controller shouldn't be doing this, it's not in charge of how things look
What is happening here? Bad merge?
Can you file an issue about this?
Well there have been reports from people who are using socket.io to know when the heartbeat is received so there are probably some usecases for it.
Shouldn't we make this "gem 'machinist', "~> 2"?
If you're not using these blocks don't have them here.
I haven't done much with adding my own work to pypi.  just as an example one time.  would it be common practice to include a section here on how to install the requirements without actually installing?  Like if you're doing dev work and don't want to have to keep reinstalling for each change?
Do we need all these spaces?
Is it a problem if `recomputeSingleDocumentChangeSet()` never had the `// Look for changes in associations of the document` loop that was in `computeChangeSet()` and is now in `computeOrRecomputeChangeSet()`? That is the only difference in execution path that I can see.
Oh gosh, I was really mislead here, so ignore this and thanks for the
reminder.
No, it should not crash.
- `*myRepo` => will call to `const T* GitPtr<T>::operator *() const`, which just does `return d;`.
- The const cast actually does nothing.
- The `Repository` constructor creates an invalid Repository, if it's parameter is `NULL`.

Everything's fine :-)
See javadoc for isBlank. It already returns true if null is given. The or "||" is not necessary.
Maybe make this optional? If we know it is ordered, it will save a lot of time not to do this step.
You should add a space between `'my-blog-post'` and `}` or remove the space after `{`.
Oops :) - be sure to change this to "mapped"
Can't you use headings in a sidebar? (I haven't tested it)
Place `// ...` after this line, because you folded the `use` statements) and put an empty line between the 2 comments.
请需要添加句号
I believe you can use `quoted_table_name` and `quoted_primary_key`.
All 2.7 tests now pass. I'm checking coverage.
leave a blank line before
Por casualidad la licencia no requiere que este archivo esté?
well that's a bit misleading.
When you reach out to `VehicleCollector` -- you're actually not using the module you just included at all. Instead, you're reaching out to the main VehicleCollector module and using its methods.

it's a bit confusing, but you'll likely want to either not include the VehicleCollector at all, and just use VehicleCollector as a "singleton" module, or you'll want to:

```
def initialize
  self.class.add_vehicles(vehicle)
end
```
Add **s**

... same below
Is it not more efficient to use replace() with 2 parameters instead of this?
What about the block below // TODO? Is it really needed? I don't  understand the meaning of source and destination here.
Can you remove these new lines?
Can we just remove this?
Hmm I like the debug message better than the one below. Can we change the exception message to say the same.
This pretty much what `mapcat` does:
~~~clojure
(mapcat (fn [[key value]] (repeat value key)) a-map)
~~~
There is a lot of tight coupling here with the axis knowing about the locators private ```_nbins``` attribute. Can you think of a neater solution (short of adding ```set_nbin``` and ```get_nbin```!).
I think you're right.  I fixed that.
Did you misspell this no purpose?
`.message` is a generic class name that is already being styled by webapp styles.  If you need this specific styling, change the class name to be application specific (perhaps `cts-message`).
Same deal as for the short drop, except that we want to keep the largest break stat. We replace `nodata` with a large negative number, do `(reductions max ...)`, then replace the large negative value with the standard `nodata` value.
Why are you sending me to edit_questionSet?
this is a bit of a odd line to read, there is a lot going on here
Gah. Forgot to commit new file .. my bad
Why is this necessary? And are you sure uname is in /bin on *all* platforms?
- Are you sure ``mt_rand()`` has the correct type as "integer" is not a sub-type of "string|null"?
- Are you sure you can call ``mt_rand()`` with the given parameters? It seems like at least the 1. parameter (``$min``) is missing, no?
we should use the enums here instead of the literal value
Zu viele Leerzeilen...
true. will do that.
Please make this 13.3.
This should be specific about what "correctly sorted" means in this case.
Okay, I left a cut at 10 minutes, and added TODO for data older than 10 minutes.
Well, because `collapsed` was initially not bound to `number_of_colums`, just to panels. So later I left it like that in case we need to add properties to panels that are not part of column layout, which seemed likely at some point. So it's just for the future, it would be a lot of work to add panel properties later.
Ehm, is not `new Date(Date.parse(post_date)).getTime()` the same as `Date.parse(post_date)`??? :-)

Please add spaces around `/`. You can also write `/ (60 * 1000)`, so that it is more obvious what is happening. Also a comment that it is converting of milliseconds to minutes would not be bad. :-)
Same. We could invite users to tell us of horoscope source she enjoys and we could try to support it.
(After I pull this in.)
Agree, but it is the easiest way. Will try to find another solution.
Please order fields in: date, language, sign, forecast. In this way they are ordered from less specific to more, and then at the end the content.

Also add indexes to date, language and sign fields.

Order those fields also elsewhere in the code like this (for example, in arguments passing to functions).
Could  you remove this if block if you made num_entries_to_read very large?
Согласен, разумно сделать выброс ошибки открытым методом
Tab at the EOL.
like here and next line, and till L141 (and some after that)
why are these defined? If it's just because of the _hibernate-search-infinispan_ option, then we should not mention it directly but rely on its dependency.
This seems like a restatement of the first reason not to have global variables. I'm hesitant to just say a thing is a bad practice without saying *why* it's a bad practice, but in fact you've already said why it's a bad practice, so this paragraph seems redundant to me.

I'd simply point out that IIFEs provide a way to avoid polluting the global scope -- ensuring that variables can't be tampered with by other code -- and move on to the example below.
Continuation indent.
Sorry I'm not quite familiar with bash script, but this line of code works on my computer. I'll change it to double quote for safety.
This will probably move to the features.inc file.
Why not `0.05` ?

0.05 × 255 ≈ 13
0.055 × 255 ≈ 14
0.06 × 255 ≈ 15
With `attrs.unshift` you don't create an additional array.
Probably a common enough pattern to justifies a `show_hide()` method on the overlay class.
Maybe even ELReg is fine here.
Cool functionality. 

Out of interest, is there any coding standards defined for statsmodels which suggests this should subclass ```object```?
self.params contains zeros...it's only self.cov_params that has nan
Agreed, blocking should become the default above a certain sample size (O(500)?).
Rather remove the code and leave the comment only? Is there a reason to leave it around?
^ see above: "Currently uses various testsuite and launcher snapshots that will eventually be staged before this is merged"
it's underneath (line 51)
But if you do want to leave in the inline style, you could optimise your code stil further by removing the semicolon as it's the last rule. You've clearly optimised for low bandwidth so i thought that extra byte would help you eek out the last % of speed.
stray # here.
Include message saying what was wrong if check fails? e.g. including the original ref argument passed in.
Remove the break here.
Also the partial uses 4 space indentation whereas the rest of the project uses 2 spaces.
I think we want to be able to allow query arguments for POST requests. Something to think about.
This has nothing to do there. The master branch of the SE is meant to be used to develop 2.2
@Tobion I know, but i think __Symfony2__ should use the latest technology. 
The "form component" already using "HTML5" technology, so we should also take the HTML5 syntax.
Yesterday pushed changes that fixed this and when I look at DIffs it is showing it as changed.
Can you call initOneCharMode from this init?
It is the way Objecteve=C expected isn't it?
This seems incorrect, why use a comma for two options?
Reassigning a variable another value is generally a bad idea.
ContentApi, article pimping and suppressApi404 all belongs somewhere more relevant than generic application context file. They are too business logicy for here. Moved to `content` package.
Then the initialisation should really be a part of the PluginManager. Just send an instance of the plugin over to an *initialize* function and set every object there.
We should remove the extra spaces from the beginning of the lines.
hi. のある行途中の英語だけを{hidden} タグで囲うつもりが、先頭から囲ってしまっていました。

テストフェーズ、テスト種別に修正します。
ちゃんと調べてないけど、1.2.2まではTestがデフォだったってことじゃないかな?
we use in the scikit pylab rather than pyplot
with "import pylab as pl"
This explanation of the curse of dimensionality is a bit confusing.
This is not PEP8 compliant: there should be a space after the comma.
Same here, do we actually that many samples?
No problem :) Fixed in the next commit, thanks ;)
Same thing on the other 2 ops.
Moved code to select dialog to method in MeasurementActivity class.
This won't work, it needs the '()' in order to execute the the close function.  close is just the reference to it.

The clean way to do this is to use:

```python
with open(template, 'w') as f:
    <use f here>
```

which ensures that f gets closed when going out of the with scope.  You don't need the finally with that construct
shouldn't this be decoded as utf-8?
All good things to consider.

Another thing we should consider is not mucking with the passed in cacheName.  This would make the caching mechanism more configurable.
This task isn't being used and it was blocking me from running grunt. Had the same issue on another project.
`degre` ?
As you prob. know, this was copy of code release in master..
Can u release a fix there (or 3.10.x) ? and I'll revise this commit in a moment
I see. Do you think I should remove it and only use it when/if I add non-virtual to ITypeInfo? Or leave it and save the effort for the next time?
To be honest, it won't save much. Return an index rather than an address is saving nothing in the final assembly code.
This was something I previously commented on..

this index memory should use rowManager memory.
It was specifically the //_forceRoot I was questioning.
We can use tenant_router created in L.152.
same as above comment about making this like the old site, could fill in the date column.

Doesn't seem worth it, we could return a totally empty document or an error page for all it matters.
Is this needed? It's hard to tell from just skimming the code due to its size but I can't see anything that it protects from running.

This is repeated several times.
Same as in #L51
personally I would rather just duplicate the above function and do `atimport() || atcharset()` etc
After another look, most likely you're right and there was a mistake in original code.
Are other values than "held" actually interesting? In the majority of cases we're trying to collect the current state. "Barrier" and "limit" probably hardly ever change, "maxheld" I think is totally useless in our case (the value can easily be reconstructed using a maximum aggregation).
All the files in scripts/jsonrpc are changed? This should not be the case.
actually lock to 0.0.4, @KAllan357 just pushed a new version which fixes some broken things
Yup, that's exactly what I meant, and how it was being used before. If you stopped playing MindSnacks all the time we could have talked about this in person. :smiley:
BLLLLLLLLLA
Use loginManager.. look at the recent update. sorry the checkin just went through today.
@shacker as we've previously discussed, we probably don't need the `.promise()` here.
`(> (author-count book) 1)` palauttaa booleanin, `if` on tarpeeton.
I thought this may be a bit contentious. It's because the `MemcacheCheck` requires memcache, and the `CustomErrorPagesCheck` requires twig. Happy to remove them if you don't like the idea.
You should print the paths to the files that need cleanup.
This lack of newline makes me feel like I need to wash my hands one hundred times.
I don't think this line is ever reached.
Well, anything typehinted as Jackalope is clearly implementation specific. Such namespace should not appear here
Wasn't the last version 2.5.0? :)
I put a lot of effort into guessing 120 frames.  lol.  Longer-term, we will have something better and more general since we need this type of logic in several spots, e.g., releasing other caches like vertex arrays for the viewport quad and ellipsoids.  I think @kring and @shunter need it for terrain too.
`vec3(0.0)` is more concise.
Should be "A point on the surface..."
Enrollment?
Yeah, indentation is botched for the fireEvents, rest is checked though :)
I would move this to a top level Storage section like Cloud Service as Storage is not a Cloud Service.
Missing space after period.

Technically, second sentence should be in description (summary should be one line).
Default is undefined but behaves as if true.
Can you get rid of the commented-out code?
Commands should be case sensitive. I wouldn't use toLowerCase().
Ok, 
considering multiple inheritance this is possibly a clean solution.

I said it's not really nice, thinking about it again, it's completely acceptable
Recommendation for this and similar "no context" failures: fail("Failure when creating remote server " + ce.getMessage()); 

This will make error reports on CI builds much more informative and not needing to directly go lookup source to grok it.
How about doing something like this instead?

```ruby
def chief_complaint_options
  treatments = if @patient.new_record?
    Treatment.all_names
  else
    Treatment.available_names
  end

  # ....
end
```

I've been using helpers less and less in my new projects and I am seriously considering doing away with them here. But until then, this works.
Does the static site generation tool we're using support URLs without extensions? The usual hack is to have foo.md produce foo/index.html, giving a path of `/foo/` rather than `/foo.html`. This is preferable.
Same for the method name here
The service_* settings are no longer relevant. Keystone used to be able to function as a proxy, but the functionality is gone, and with it went the need for the service_parameters.
You know what they say, it's better to do it twice just to be sure! </troll> :tongue:
According to "[Remove preg_replace /e modifier](https://wiki.php.net/rfc/remove_preg_replace_eval_modifier)" you should not use `e` modifier.
The solution should be TOTAL_SALARY
So the original code had the declaration of the two structs down in the set of version blocks (i.e. line 2988). The old appender only used arrays, so all it needed to know about was a pointer and the type's runtime type information. The new appender contains manifest constants derived from the sizeof of the elements. So in order to instantiate the appender template, it needs to know the size of the element type. Here are the specific error messages:

std\file.d(2978): Error: struct std.file.DirIteratorImpl.DirHandle is forward referenced when looking for 'sizeof'
std\array.d(1883): Error: struct std.file.DirIteratorImpl.DirHandle no size yet for forward reference
std\file.d(2971): Error: Appender!(DirHandle[]) is used as a type
oops. That is what I meant.
It does work:

```d
import std.stdio;

struct S
{
    int x;
    bool opEquals(ref const S s) { return true; }
}

void main()
{
    S s1 = S(1);
    S s2 = S(2);
    writeln(s1 == s2);
}
```

And it works with `int` as well. So I'm saying, apart from the doc fix we should also probably fix the compiler.
Hmmm, that's weird. The test can't be compiled in Eclipse (it does with javac), e.g. on line 197:

    Cannot cast from Set<AnnotatedConstructor<capture#11-of ?>> to Set<AnnotatedConstructor<?>>

I can fix that by adding a type parameter `T` to the test and using that for all the mocks. Then the test also passes in Eclipse, but it fails when running on the command line. I guess I'll wait for your other tests and see how it looks then.
非同期プログラミングの一つの問題として、エラーが別のスレッドで起きる可能性があるため、伝統的な例外処理であるtry/catchが
Le `use` est juste déclaratif, donc, ça marche à la compilation, mais si lors de l'exécution, les classes qui en dépendent ne sont pas disponibles, PHP générera une erreur.
Also, I encourage you to wrap this in a catch block, or otherwise ensure safety when this throws. Because Android.
Are we trying to keep Sync setup stuff out of the task list at all times?  Can we have a comment explaining what the plan is here, and possibly normalizing all the activities?
Cursor won't be null here.
We'll need to follow the terms of original MIT license.
I don't think that the guideline actually defines strict rules on naming internal variables. That variable is human readable, is descriptive and is not exposed to the user, so I do not think it is worth renaming.
Also, this exception also doesn't provide any information about where the instantiation really failed - you should provide user with what Page Fragment it affects.
You might wanna add some Capy checks in dashboard_helper_spec.rb around line 96.
On a second thought, this will riddle the error console with exceptions. Although undocumented, returning "" instead is just fine
OK, so you just want the region parameter and not the modifications to the endpoint?
.gets(nil) would read everything (no separator) -- i'm not sure why you wouldn't want to just do .read, though?
I don't think that GML & WMSGetFeatureInfo are used nearly enough to be included in any light build.
Is there a reason why we're doing this for `-webkit` only?

aka why aren't we adding `-moz-user-select: none;`
This is the most important code line.
The code will trigger on all segmentation faults.

My gripe here is that the code throws NullPointerError for any dereference of a pointer in the range 0 - 64k. A null pointer is a null pointer. It is not 1, 2, 3, or any other integer value. Ever. The code should throw NullPointerError for 0 and 0 alone. Any other pointer should result in InvalidPointerError.
Yes, but that is an artificial region . .  a column in the table. . . and notice there is no mutual exclusivity and that is what I am struggling with. We may end up going that direction anyway, but I would stay away from it for M1. . . that is all I am saying.
I hate mutual exclusivity ;)
Can we remove the "Storage" instances?
fixed in the update
General comment: always write out all arguments a callback function can get. In this way developers after you will know what is available.
Yes, it is very important in this context. In my opinion, since there's a link to the actual documentation on the subject, I'd say a recap is unnecessary.
On Tue, Feb 28, 2012 at 12:32 PM, Miguel Cabea <
reply@reply.github.com
> wrote:

> > @@ -137,7 +137,11 @@ def compile(pattern)
> >
> >    public
> >    def match(text)
> > -    match = @regexp.match(text)
> > +    begin
> > +      match = @regexp.match(text)
> > +    rescue
>
> I guess one could rescue only ArgumentError exceptions in this case.
>

That'd be safer, I think. Catching all exceptions is noble, but you end up
hiding real bugs (like this UTF-8 one).

Still, I don't know if this is the right solution to the invalid UTF-8. :(

I mean, I agree with the idea and the code, but I wonder if there's a
better way to solve this? Like, why do we have invalid UTF-8? If we're
really processing binary or ASCII data, we should set the encoding to ASCII
or BINARY, I think? Hmm...

>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jordansissel/ruby-grok/pull/3/files#r498281
>
Is your concern that the client would be released but fails to be because there are outstanding connections that reference the client?  That seems unlikely to me, because clients are long-live.... most likely the life of the application's process and the connections are short lived--they are released by the tables as soon as the connection finishes.  And connections have default timeouts of 1 minute.

I don't think this is a problem, but we can discuss more if you'd like.
by **the** Bean Validation provider
why are we notifying of a message from inside the chatbox?
lots of "stair stepping" here. maybe could simplify and just have 1 `try` block and catch appropriate exceptions at the end? ie:

```java
try {
  HttpService http = serviceProvider.getHttpService();
  ...
} catch (MalformedURLException e) {
  ...
} catch (AccessDeniedException e) {
  ...
} catch (MalformedURLException e) {
  ...
}
```
@arthurevans actually his comment is talking about a specific flag that changes LOGD in the tiapp.xml.. I agree though that the wording is confusing (and actually incorrect)

@hieupham007 The actual property name in tiapp.xml is `ti.android.debug`, and the wording should probably look something like:

`
/**
 * Whether or not debug logging has been enabled by the Titanium application. This is normally set in the application's <code>tiapp.xml</code> with the 
 * <code>ti.android.debug</code> property like so:
 *
 * <pre>
 *   &lt;property name="ti.android.debug" type="bool"&gt;true&lt;/property&gt;
 * </pre>
 */
`
Can these two be combined into the `@return` statement like below?
On Fri, Mar 2, 2012 at 12:09 AM, Ken Barber <
reply@reply.github.com
> wrote:

> > @@ -15,3 +15,10 @@ def get_root_home
> >  Facter.add(:root_home) do
> >    setcode { Facter::Util::RootHome.get_root_home }
> >  end
> > +
> > +Facter.add(:root_home) do
> > +  confine :kernel => :windows
> > +  setcode do
> > +    ENV['ProgramData'] or ENV['LOCALAPPDATA'] or ENV['TEMP']
>
> So what is the decision here - I don't think root_home should represent
> anything but the original intention - aka. the root users home directory.
>

I'll just pull root_home out.  I can't find anything that needs it for
Windows.

-Jeff
do you plan to throw them youself? if you like to be called define it in the API; you're called and does not retrieve events in that kind
`s/write/writ`. I know, I know. I don't make the rules.
Вообще сейчас идет "тренд" Event Driven Development (Loose Coupling) когда все приложение построено на событиях. И модули вместо вызова метода другого модуля общаются событиями http://habrahabr.ru/post/123635/
why do you need this?
If the state is interactive shouldn't you wait for it to be complete. What if they haven't subscribed to webworksReady yet?
how does this work?
What if I want to send in HTML body?
can it handle it?
"retrieving" typo in comment
You already have these elements saved as the variables above
``*DEPRECATED*``
This lets you point to a different Well model than Armstrong's without having to do multi-table inheritance. Seems useful.
url encode activityId
Quoted, please.