labofsahil/github-event-dataset-2012 huggingface dataset:
ApplicationPool/Client.h and ApplicationPool/Server.h are going to be removed in 3.2. It's not necessary to work on these files.
I would prefer if this called `this.postParameter(name, Boolean.toString(value))`.
We have discussed this with Harut (because he has implemented ImageView)
2 is an error, I agree. But what would you gain by moving the check for one further up? You still have to count the modules for that position.
Instead of using a curl specific option, why not use a generic name like the stream options (follow_location or max_redirects) and then at the transport level map them onto their respective options. This will automatically work for stream as it passes in the HTTP options straight through and cURL will require some level of custom handling either.

Safe mode is gone in PHP 5.4 as well.
Updated haystack?
Is this the right documentation (the parameter is named type)
Find plugins containing marked Vaadin-Resources(in this case is plugin ......resource)
yes, unpublishing something with the date based publishing model would be handled by having the end date smaller than today. with all respect to that client, i think the ux of the date + checkbox model is confusing as its easy to produce conflicting information. if you never want to publish you could also set the start date to null and handle that as never published.

as for the acl question: that comment was meant for the checker, not for the interface on the document. the checker is doing exactly the same whether its about workflow or something else - it has no interface method to handle workflow transitions or anything. i totally agree that the implementation and thus the interface for documents is specific to workflow. but i propose we call PublishWorkflowCheckerInterface AccessControlInterface. but if you see drawbacks in this we can drop it, as you are right that ACL's hopefully will be implemented one day in jackalope. RE: Symfony2 ACL system: do we even need this then or could we hook into the ACL?
`user_address` -> `email_address`
I think this should not be necessary?
Object/dict keys should be written as string (`'src'` instead of `src`).
We don't have space before `:`. Why space?
If you select multiple files and then double-click, it should open all of them, not just the last one.
I guess you could do:

     options[:credential4].presence || account
Docblock? Should be sure to reference all the possible string constants that can be used for `theUnits`.
prefix with `\` as you're in namespace `DoctrineModule\Authentication\Adapter`
double nitpick: although _load_catalog seems a private member, it is imported. you can import in_data_dir and _load_catalog with a single statement
The method should be implemented (it comes from TestHelper) but I didn't know how to exactly do it for the other TestHelper
There are no SQL schema generated for every packetfence version, it's only if the schema was updated so this, while clever, won't work.
I think I may have broken this.. Each submodule will interpret `${basedir}` relative to their own directory. We may want to move this systemPath entry directly into `elephant-bird-hive/pom.xml` for the time being.
"callback" is not too descriptive, perhaps something like $sourceProvider?
Shouldn't be related to the term?
window.messages
Looks like this could be in an outer scope.
Should use your own `removeClass`
Why not use `UNI_BIT` here?
On windows they might not be collected by dependency checker when the .dll is loaded by ctypes, is not available on PATH and manifest is not included. This is seen more often.   On windows dll files should be handled as data files. It is not true for mac. Windows only.

So let's summarize it: ignore extensions   .py, .pyc, .pyo, pyd, .so, .dylib
i will fix it before merging
Yep, you're right.  Will fix and push in a few min.

On Sep 14, 2012, at 2:51 PM, Mike Jarvis wrote:

> In include/galsim/FFT.h:
> 
> > @@ -51,6 +51,7 @@
> >  
> >  namespace galsim {
> >  
> > +    //! @cond
> This is a bit cryptic when seen out of context. Perhaps a comment each time (or once in each file) you use this saying that the code between @cond and @endcond are excluded from the doxygen output.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 


-------------------------------

Rachel Mandelbaum
rmandelb@andrew.cmu.edu
http://www.andrew.cmu.edu/~rmandelb/
Here's a quick example I ran in Python (for finding the half light radius as a multiple of the scale radius):

```
In [1]: F = 0.5

In [2]: def riter(rin):
   ...:     import numpy as np
   ...:     return np.log( (1. + rin) / (1. - F) )
   ...: 
```
That's the cumulative flux and iterative function set up.  Then (making a fairly off-beam guess)...
```
In [72]: r = 0.1

In [73]: for i in range(30):
   ....:     r = riter(r)
   ....:     print i+1, r
   ....:     
1 0.788457360364
2 1.27450061899
3 1.51490770073
4 1.61538328393
5 1.65455783863
6 1.66942528226
7 1.67501037971
8 1.67710044117
9 1.67788146444
10 1.67817316413
11 1.67828208748
12 1.67832275741
13 1.67833794238
14 1.67834361194
15 1.67834572876
16 1.67834651911
17 1.6783468142
18 1.67834692437
19 1.67834696551
20 1.67834698087
21 1.6783469866
22 1.67834698874
23 1.67834698954
24 1.67834698984
25 1.67834698995
26 1.67834698999
27 1.67834699001
28 1.67834699001
29 1.67834699002
30 1.67834699002
```
In tests I found that r got close to being good quite quickly, but then took a rather long time to nail the later decimal places.  Of course, there are a million ways to solve roots, but it immediately struck me that this equation was simple enough to be able to construct this very basic iterative solution.
> Please check the edits that I just pushed.

That's fine.  I guess he details of this will be expanded upon in #248.  So that's probably good enough for now.
OK, I also don't think it always fits on the page in a way that looks nice.  I'm not going to take action on this immediately in that case.
Should put a not in about the accuracy obtained with these limits: 

1.01 -> 0.4966666 versus 0.4966898
0.99 -> 0.5033333 versus 0.5033568

both have fractional error of 4.7e-5.

I don't know if we'll ever care about errors of that level, but switching to 0.999 and 1.001 drop it to 2.3e-7 fractional errors.  Might be worth doing just for the heck of it.
I wanted it to look exactly the same as materials pages and I didn't want
to spend a lot of time on styling. So I copied. But you're right that is
hideous HTML.

Will check if easy to get right while addressing @jinpa's comment.

-- movile
On Dec 12, 2012 1:29 AM, "Jason Bau" <notifications@github.com> wrote:

> In main/templates/content_group_list.html:
>
> > @@ -0,0 +1,30 @@
> > +{% load url from future %}
> > +
> > +{% if downloadable_content %}
> > +    <!-- child item block -->
> > +    <div class="button-tray" style="display: block; clear: both">
> > +    <h3>Downloadable Content for {{video.title}}</h3>
> > +    <ul class="unstyled course-list">
> > +    {% for child in downloadable_content %}
> > +        {% cycle 'zebra_light' 'zebra_dark' as stripecolor silent %}
> > +        <li class="course-list-item {{ stripecolor }}">
> > +            <span style="float: left;"><strong>{{child.title}}</strong> as {{child.name}}</span>
> > +            <span style="float: right;">
> > +                <a href="{{child.url}}">
>
> I don't understand why the button inside an anchor tag...
> can't we just have <a class="btn btn-mini"> ? Is the button better html,
> in your opinion?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Stanford-Online/class2go/pull/1477/files#r2390341>.
>
>
ObjFlags uses an automatically-created field called 'uid' for its key. You are specifying that the key is a number that you will provide. It must be specified. Otherwise, the 'uid' field is added and used as the key. The problem may not show up right away, but could in the future.
Why only select the first host?
Thankfully, installPackage doesn't modify the map arg so you can use ImmutableMap.of(...)
Ah, I see that setBlockingDetails calls `withBlockingDetails(description, null)`. I think that just makes things confusing, and error prone for a user who calls withBlockingDetails with an accidentally null task.
Good to always use `\@AfterMethod(alwaysRun=true)`, rather than specific groups - I think that behaves more as folk would expect unless there's something really specific (which there normally isn't with `Entities.destroy(app)`). The before/after methods won't be run for your live tests. Not that they currently do anything :-)
Looks the same as Predicates.equalTo(0)
@minggo 
for cocos2d-iphone, I am using the "tests" prefix for all the tests.
Could you do the same for cocos2d-x + Android ?
No idea, it was unclonable before I modified the handlers and didn't want to make it clonable without a reason.
I suspect it is because in 1.2 it used to assign the connections differently and without a clonable handler it would screw things up bigtime.
This could be obsolete now, but I'd rather not change it at this point in time :]
actually , it's the whole transaction that gets aborted, so maybe _abort_handling ?
How would that work? You should also update the config.php and check.php to use your requirement class, which won't work as we also override them.
This feels strange having the notification app as part of the reviews app.  Can we create a new ``NoticationsApplication`` class instead to keep these separate?
Naming is inconsistent. Here is means higher than 10 (exclusive 10) and in line 104 is means inclusive 21.
Btw, these test still fail on my windows. I get higher values, e.g. 31.4.
There is no need to add parenthesis here.
This file has been moved. Look under app/views/. Get the latest version and redo your changes.
Add this file to your .gitignore file.
Nope, that's all you need to do! Notice the "imjeee added some commits" at the bottom of the pull request.
Readd these 2 lines :P
There are a reason we had these in in the first place.
add spaces around `!=`
consider response.raise_for_status()
Would look better if used with sizeof
I think this might be better to use both VID/PID.  For example the VID we use is a large hobby block and might grab irrelevant devices.
I don't mind copyright information et al being added, but I don't think you own the copyright on this code.
I appreciate the effort and reuse of existing patterns, but, as the author you need to figure out what this, and what every line in your PRs are for.  Copy/pasting code without understanding is not acceptable.
really? I thought it was the merchant_admins?
Вы же написали, что нужно сделать компаратор отдельно и для строк, а не только для пар, чтобы сократить код.
Any reason you are doing this here if it's not internationalizable?
Why did you remove head?
space above here, for PEP8 hotness on imports
add `public` keywords (PSR-1/2)
I changed the naming convention to AuthenticatedMember. If this isn't ok, I can change it back.
I agree with the types, I was wondering about the `selected_add ` optim.
Fixed in 	 57031c8
I don't mind, I am okay with that.

While I am at it, I must warn you  though, I am currently making *huge* changes on the tree structure (see #946). I don't know how we should resolve our future conflicts :/
I'd say that it is hiding another issue... The algorithm should work without enforcing positivity.
It is.
It seems redundant with the print function from io.h.
а конкретніше? які проблеми і помилки
This does not look consistent with the other places using prototyped nodes in Symfony
as I understand, it iterates over the result of of xdebug_get_code_coverage(). The result is an array with $fileName => $data, where $data indicates if this line is covered (1) or not (0). 

But I have no idea why there is a check on gt -2 and I didn't find anything for it in the web. I found out that in older versions of the xdebug-module the value was incremented for every time this line was called, but currently the value is afaik just 0 or 1.

I have to investigate this a little bit more.
According to my simple test, GetItemFiles (false) returns only the .proj file. Moreover, neither GetItemFiles (false) nor GetItemFiles (true) returns the folders the project contains.
Actually, if you're going to the trouble of searching for type mismatches, I reckon it'd be brilliant if you can find which parameter is missing, at least for the case where it differs by 1?
(int, float, char)  vs  (int, char)   --> Parameter #2 ('float') is missing.
Only use the "parameter count mismatch" error if it's ambiguous  (eg, (int, int, int) vs (int, int)).
It's a bit of work to do this, but it's actually what you do in your head when you get this kind of error message.
Let's try to avoid whitespace changes in future, to minimize diffs.
According to API doc - If offset and length are omitted, data is written starting at the beginning of the buffer. Probably we don't need to throw exception here?
This will keep adding new connections, need to make sure it is only done once.
This is a `TODO` for when we do Multi-part from POSTs.
err. no. that is an artifact of starting down one direction and changing course of the design.
Also, shouldn't migrations use [`Doctrine\DBAL\Migrations\Configuration\Configuration`](https://github.com/doctrine/migrations/blob/master/lib/Doctrine/DBAL/Migrations/Configuration/Configuration.php) and not just a plain array?
will a static cast work and is any cast needed at all?  I would have expected map.mapToValue to return a ReferencedFile
It should spill here if:
if (!disabled)
{
    if (isAllDisk()  OR (isAllDiskOrAllMem() AND hasRowsOnDisk()))
    {
    }
}


>Is the canSpill() required now that the collector->setup parameter has changed?

In the mini sort case, it isn't strictly needed anymore, as that also corrected to specify type rc_allMem, an therefore couldn't hit this branch anyway now.
But in general needs to check if disabled.

I'll make the if disabled a separate line and add a comment.
Решила пока не хвататься за грозный Devise gem и ему подобных товарищей? А чего так? :smiling_imp:
а навіщо їх стільки?
This can be simply `$e->getViewModel()` -- $e is already the `MvcEvent` instance at this point.
Add support for QEvent::LanguageChange
To highlight these important low-level modules, I'd like to have a task-based tutorial, like "Displaying a Popup" and all the other here, that describes some of the things people need to use these modules for. For example, maybe, "Listening to chrome events" or something.

In this new structure I no longer split the tutorials into "Developer" and "Internal": they are all together, and I'll flag each tutorial if it uses low-level modules.

So if you can think of important things people want to do, that need tutorials and require low-level modules, go ahead and raise a bug for it.
The active property is defined in "AbstractPage" and the value is also "false".

https://github.com/zendframework/zf2/blob/master/library/Zend/Navigation/Page/AbstractPage.php#L134
Conflict. Info, this shouldn't be here
It does not work! I don't know why (and I don't have time now to search for), but the test fails in both cases you presented.
Hmm. This feels odd, since we now embed the environment-to-XML mapping in two different files (Gemfile & this Rake task), but I don't know a better approach yet.
Do you need to check if the user is already in a request here? Otherwise you could end a request before the user wants to.
I'd say this isn't really necessary
Metric class is used to represent the metric name today. In the future, we might expose some metadata such as units for the metric etc. So your decision sounds good to me. I am a little lost about your check -> many agents mapping. I don't think the mapping is valid.

@jirwin: I think the driver field is not extraneous when you want  to pipeline results. I forgot the details but I think LazyList might use it.
Never do this.  I know there are lots of examples in the framework that rescue all exceptions, but this is very bad practice.  It will mask syntax errors and all kinds of other ills.
dprintf prints debug to log file (/tmp/meterpreter.log.PID) only if debugging is enabled on meterpreter payload (which isn't by default)
it's invaluable when developping and there are this kind of debugging statements everywhere in posix meterpreter code
how about href="#advanced-search-options". Since a lot of browsers show the link text, it seems to be better UI to show something that users understand even if it still meaningless.
if we're talking compliance this isn't really valid either then
Hmm, this seems to indicate that if-else is faster: http://jsperf.com/switch-if-else/13 I wonder how it changes with the number of cases.
I think we can get rid of the | 0 on this line, because adding three integers will yield an integer.  Hopefully @shunter will chime in if there's some subtlety I'm missing.
I'm fine with whatever you decide, but I'm not sure I understand the reasoning.  Me and @shunter were discussing this and the way we see it, the only reason Context includes TextureAtlas is for this method, so if an app isn't using TextureAtlas, it is still required to include the TextureAtlas code.  Why not just make people construct TextureAtlas directly, this way they eliminate the code from their deployment if they aren't using it.
Please hold on bumping the version number
Whole bunch of trailing commas above.
Also spaces around the "else" keyword.
Is there a reason this needs to be synchronous? We should make it async unless it needs to be synchronous.
require.on instead of addEventListener
After testing and consulting with Opie, confirmed that buffers should always be mutable.
Afair, this was a copy/paste on how to get dependency on the needed log handler service.
could be better called document_field.
This makes sense.  I'll work this in.
Валя под всякие UI-штуки завёл папку `ui`, но я пока не проникся её целью. Возможно, кроп как раз туда должен пойти.
pull requestしてからでアレですけどメソッド名，load_pluginにしたほうがいいですね
Great, I'll fold it in tomorrow.
Wasn't able to reduce this one, unfortunately…
I'm not sure what you mean. How is this destructive?
here you may want to test whether using `&` instead of `&&` generates faster code
This is not i18n friendly. Please update.
The purpose of incremental flushing is to get stuff out of memory and on to disk. I don't see any point in bulkInserting only some of the stuff we have queued up, particularly if we are calling `incrementalFlush` on every possible insertion. The only time we'll be above `flushThreshold` is if we queue up a ton of records and then find their parent, and that's (a) pretty unlikely, and (b) we're probably better off flushing the whole lot rather than allocating a new `ArrayList` and a bunch of subList references.
This seems strange.  I know this was causing trouble, but this should be done by the db.  Unless you don't want to send the encryptedUsername in as an arg?
Switch these two statements so that the postincrement is the last operation? Or just add a third statement to increment.
that needs to be nested in `akka.actor` to take effect
Well, the alternative would be to hook the right delete listener to the right type of entity (object or annotation). This would allow the annotation system to delete the annotation and refresh the (comments) tab instead of relying on the comments system to remove the deleted annotation entry from the comments tab (without a refresh).

Side Note: If the current object-based annotation implementation would listen for object deletion events (because annotation object can be deleted from a page by the user), the problem would be fixed as far as deleting annotations as comments is concerned.

However, if we would be to respect this 100%, we would need to do the same for the edit tool and that would be a bit more complicated, since the edit form is generated by the annotation system using a customizable page. I`m not sure how well this would work with the comments tab.

Why don`t we keep this for future iterations?
How is this going to work in reality?

When they give us patients, do they give us every single patient ever and tell us which ones have been assigned?

Otherwise, this test doesn't really do much for us.
yeah .. though i would make it 10 or so .. also i can never remember if it needs to be negative or positive to run first :)
It does not look like int and floats are converted to their respective types; it looks like they are left as strings. This means that if the properties are read out and assigned to a new message, they will be sent to the server as strings the next time, changing their semantics.

Perhaps you can use your regular expressions to interpret only digits as integers, and only digits with a decimal as floats. Won't be perfect, but should get the 90%.
You probably didn't mean for this to be here.
These should have default values, so we don't have to do `isset()` checks later in the extension class.

```
->scalarNode('retry_connect')->defaultValue(0)->end()
->scalarNode('retry_query')->defaultValue(0)->end()
```
ok. if somebody comes with a use case, its easily changed. and anyways the odm currently does not prevent you from persisting mapped superclass documents...
I think GFI is used more than GF, but we could go ahead and include them
both.
Yes, popup is certainly required (I'm just so used to using geoext or
jquery popups that I forget about the standard openlayers popups)


2012/2/28 Éric Lemoine <
reply@reply.github.com
>

> > +OpenLayers/Control/WMSGetFeatureInfo.js
> > +OpenLayers/Control/Panel.js
> > +OpenLayers/Handler/Point.js
> > +OpenLayers/Handler/Path.js
> > +OpenLayers/Handler/Polygon.js
> > +OpenLayers/Renderer/SVG.js
> > +OpenLayers/Renderer/Canvas.js
> > +OpenLayers/Renderer/VML.js
> > +OpenLayers/Format/GeoJSON.js
> > +OpenLayers/Format/KML.js
> > +OpenLayers/Format/GML/v2.js
> > +OpenLayers/Protocol/HTTP.js
> > +OpenLayers/Strategy/Fixed.js
> > +OpenLayers/Strategy/BBOX.js
> > +OpenLayers/StyleMap.js
> > +OpenLayers/Rule.js
>
> If the build includes tools to do feature selection/query (SelectFeature,
> WMSGetFeatureInfo) then I wonder if we should also add Popup.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openlayers/openlayers/pull/254/files#r497389
>



-- 
Matt Priour
OpenGeo - http://opengeo.org/
Expert service straight from the developers
These should be internationalized (some other places in this file as well)
You don't really need this `function`. Just a relic from the good ol' days.
Yes, `GC.free()` takes a `void*`.
I don't think we can safely rely on hashCode() - if they are not using a CCF we'd be relying on the vendor's Session's hashCode() implementation to ensure uniqueness. 

I think you need to use ObjectUtils.getIdentityHexString() - which ....

    /**
     * Returns the same hash code for the given object as
     * would be returned by the default method hashCode(),
     * whether or not the given object's class overrides
     * hashCode().
     */
These injectsions do  not work for me if the doctrine_cli alias is used. However, they work fine if a class is used. (Also had to add the `'add' => array( ...`)
If you don't mind, that'd be useful; the game does work in IE 8 as-is, FWIW.
Test coverage shows lots of important looking code in ShapefileReader is not tested by ShapefileReaderTests.
I like the per marker total display; it will help distinguish between the types!
Please move the xsl:for-each into an xsl:template (you can use the mode attribute to have several types of templates per branch and it is recommended to use it in this case). These are faster than xsl:for-each.
Definitely should:)
<string name="time_lapse_title">縮時攝影</string>
This will be updated when they(xunit-html-cov) fix their package right ?
Purchase is not defined anywhere
I will call success() here as you specified. The callback will happen later once the "payment.purchase.callback" event is triggered.
I chose "fakes" based on http://martinfowler.com/articles/mocksArentStubs.html . Either one would be ok with me.
done-added
Not sure is ```/files``` is acceptable in this case. Should I put an ```assertex()``` here, too?
Should this be persistent-dir?
This should be calling ``filter_and``.
Found that the links were including the line break from the file, and thus breaking the images that were present. Replacing all line breaks with nothing solved this issue.
и что такое next?
Такая реализация лучше переносит пустые интервалы. Раньше у меня быо что-то типа return ++it;
ignore this comment, didn't see it below.
The cached key needs to be reset when passcode changes, since it is generated based on the passcode or the lack of it.
An include for the stylesheet, see explanation on it.
you should update it to use ``setDefaultOptions`` as `getDefaultOptions`` is deprecated
Данунафиг.
Теперь при каждом новом посте мы будем прогонять весь тред?

Мало того,  сама markDel находится внутри цикла

for(i = 1, j = 0; i < len || j < len_; i++, j++) {
These two flag were left over from when I factored the screen pushing / popping states into the screenObj below. The purpose of the screenObj.pushing / popping flags are to throttle the showing and hiding of the invocation screen in case the share link or cancel buttons are clicked consecutively. I'm going to throw these out and fix up the throttling code using the screenObj wrapper flags.
Erik explained offline that because the toastId is invalid there is no end user impact to these listeners not being removed.
@jeffheifetz good point, I'll change that
Yeah, I also need to add a translation ... for the Reset button ...
OK, done! No worries. Makes sense.
`if` on tarpeeton. Vertailu `(<= 13 age 19)` tuottaa jo halutun totuusarvon.
Why ? Will it always works for JmsDiExtraBundle  https://github.com/symfony2admingenerator/AdmingeneratorGeneratorBundle/pull/224