labofsahil/github-event-dataset-2012 huggingface dataset:
I would keep the q one - problem with k parameter and model selection is that it is much faster to train for k=1...max_k in one go and get all the results at the same time - not sure how one can make use of that in shogun's modsel procedure.
pleae use { } around the first for too and please indent with 1 tab per level
please don't use $p_ in parameters names.
Sorry 'bout the whitespace changes, bros.
maybe this should be unless development as we should be testing async in staging
The macro variables should probably be wrapped in brackets.
This change isn't really related -- I just noticed we were checking for "overwrite" twice, here and in the call to the runtime-config routine.
Is it possible to add a ``@param`` annotation for parameter ``$method`` which defines a more specific range of types; something like ``string|array``, or ``null|MyObject``?
Should the type for parameter ``$class`` not be ``string|null``?
Are you sure that ``$eventArray`` (of type ``*``) is traversable? Please also check line 221.
*CFBridgingRelease()* & *__bridge* are functionally exactly the same, so whichever you prefer. Personally I think ARC code should not have any mention of *release*, but that's just me.
how about this:

    dynamic document = HtmlPage.Document;
    document.getElementById("main");
    document.querySelector("#main");

I don't have a Silverlight/Moonlight compiler handy but from memory that should work. If you have VS can you do a quick test? If not I can try to check it later.
You can remove this file.
Did you forget

    JFactory::getApplication()->input

?
Agreed. I usually create a static helper class with stuff like this.
Ah, my mistake, it's adding every valid token, not just every token.
If this method isn't used, it could be simply deleted (not commented)
As before, ratios should probably be changed to "evaluaciones"
I think the latter is the more common method of emulating private/public functions in JS.
`self.instance` is what you search for (post object).
Didn't we say `jquery-ui.js`?
Probably also make it some top-level constant (in `piplmesh.frontend.tasks`?) you then add to template context and use here?
Yes. See the "diff" tab of pull request.

But I like the idea of having that in a separate file. This is good to make things modular.
Decide. Do you use `.set` or not?
Is not this simply all input form elements inside `form_fields` class?
There is no point to support outdated and relatively slow version of  PyPy.
The 'time' param was changed to ISO-8601 from v4 and forward. I'll add v3 compatibility for completeness
This looks really similar to
CString CGit::GetShortName(CString ref, REF_TYPE *out_type)
in CGit.cpp.
Do we need C&P the code?
absolutely agree. it was not fun writing in support for both, but wanted to
keep suit with the other scripts



On Mon, Nov 26, 2012 at 10:16 AM, Jai Ram Rideout
<notifications@github.com>wrote:

> In scripts/subset_biom.py:
>
> > @@ -0,0 +1,99 @@
> > +#!/usr/bin/env python
> > +
> > +from biom.parse import get_axis_indices, direct_slice_data, direct_parse_key
> > +
> > +__author__ = "Daniel McDonald"
> > +__copyright__ = "Copyright 2012, The BIOM-Format Project"
> > +__credits__ = ["Daniel McDonald"]
> > +__url__ = "http://biom-format.org"
> > +__license__ = "GPL"
> > +__version__ = "1.0.0-dev"
> > +__maintainer__ = "Daniel McDonald"
> > +__email__ = "daniel.mcdonald@colorado.edu"
> > +
> > +try:
>
> I think we should either port PyCogent's optparse additions over to this
> project, or use standard optparse, but not provide support for both. This
> makes it difficult for us to maintain and test, and changes made to one
> option have to be copied over to the other, which is error-prone.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/biom-format/biom-format/pull/55/files#r2224641>.
>
>
Should there be an assertion here for making sure the store is configured with a user?
Is this because the platform doesn't currently allow retrieving multiple ACLs by key?
You can handle the particular case within the constructor of WithMagnitudeScales.
Not a huge fan of using this unless we want to pepper our code base with it.  UnsupportedOperationException is basically the same thing and more "standard"
Since this plugin uses tornado, maybe we could catch the importerror and tell the user that he needs to install tornado v x.x or something -- and also add a note in the doc
Also, m-c coding guidelines tend to insist that formatting should look like

   try {
      ...
   } catch (error) {  // whitespace after `catch`
     ...
   }
Would use string property names and put them in order, so: `var typecode = {"text": 1, "datauri": 3}[contentmode];`
It might be painted a bit differently, but the end result is the same.
Is 45 correct here? The prefixed one is 45, so it should be 135 if I understand correctly? Further down, -45deg is unchanged in the unprefixed too.
Added comment:

    RewriteOptions uses static initialization to reduce memory usage and        
    construction time.  All NgxRewriteOptions instances will have the same      
    Properties, so we can build the list when we initialize the first one.
space before and after <<
F-stat --> F-statistic?
I made this class abstract but I'm not sure if this makes it mixin?
Missing file header.
You could make your awesome code even neater by using alt=alt?0:1; here.
I didn't see the need, but it would be easy to add that functionality.
I think so. sys.exit() isn't so bad as (I forget, sys.__exit()  ?). I believe it will properly close your filehandles and garbage collect your objects - and it makes apparent to readers that no other computation is happening after this point. :)

The sys.exit(0) at the end is a bit pedantic, but I wanted to make the story easy to follow.
`survey-checkbox-list` id for the ul and the 'name="survey-checkbox"` are required as hooks for the JS.
I don't need to parse the Latex expressions.  I just need to detect the _possibility_ of its existence.  If I'm too aggressive, the downside is just another js library load.  Since having delimiters $$ or \[ is a necessary condition to doing mathjax, checking for those won't miss any cases -- but will have false positives.  I think that's the fail-safe behavior I want.


On Dec 15, 2012, at 9:30 AM, Sef Kloninger <notifications@github.com> wrote:

> In main/c2g/models.py:
> 
> > @@ -1667,6 +1667,16 @@ def past_all_deadlines(self):
> >      
> >          return datetime.now() > compareD
> >      
> > +    def load_mathjax(self):
> > +        """uses a regexp to figure out if the rendering of the exam needs mathjax
> > +           the regexp are rough, but should not have any false negatives.  (at
> > +           worst we load mathjax when we don't need it.
> > +        """
> > +        if re.search(r"\$\$.*\$\$", self.html_content) or re.search(r"\\\[.*\\\]", self.html_content):
> What do you think about using a regexp like this?
> http://stackoverflow.com/questions/8484945/regular-expression-for-matching-latex
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
Hello Nico.
I suppose you create a new range and select it to prevent the selection from being lost. Wouldn't it be better to get the current selection in the timeout Aloha.getSelection().getRangeAt(0) and then select that, because the selection may have changed when the timeout event fires.
This should be the name of the RPC that was issued (probably "getMotd")
It might be nice to include some documentation here specifying that this iteration can block if some element is claimed and not released, and that it does not guarantee any particular ordering over its elements.

It might be safer to add some sort of optional timeout to the next call (not sure that's allowed by the iterator protocol, though), so that you can avoid potential lock situations in multi-threaded programs.
out of interest, why do this in a separate thread?  is there a case where download blocks for a lot longer than 60s ?
Oh, so THAT'S where internals box from backpack turned into empty box.
You can embed the if inside of the capture, so you don't need to do the capture 2 times. However I think this is temporary code that has already been removed so it is fine to leave it as it.
This isn't accurate, it's the name of the subsystem and there's no `condition` parameter.
That should probably be %d as it is the length of the servername, right?
HTTPPreconditionFailed() would be better, and with the reading of the Last-Modified header
I'm not sure we want to ship the documentation as well. any reason for that?
Then the existing impl is fine.
Format comments in YARDOC style
I prefer to fail hard, though some warning to the log file is probably useful in this case.
Updated. (added to the bottom of the ui.init.  I didn't know that function existed. Awesome!)
i got rid of this PR and replaced with a better one witha better git history

Now I'll look at the actual comments

On Wed, Feb 22, 2012 at 9:35 AM, James Bergstra <
reply@reply.github.com
> wrote:

> >              if self.async:
> >                  # -- wait for workers to fill in the trials
> > -                time.sleep(self.poll_interval_secs)
> > +                time.sleep(self.poll_interval_secs)
>
> can you make your editor stop putting in trailing whitespace!?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jaberg/hyperopt/pull/82/files#r475900
>
"...adding the Hibernate family of projects" ... <- missing some articles and prepositions.
Please get the latest copy from repo and apply your changes again.
Please add the namespace "t:IsCancelled"
[(((world.time / 600 % 60)/10) > 1 ? world.time / 600 % 60 : add_zero(world.time / 600 % 60, 2))]
that'll do it.
`snow_leopard_64?` is deprecated; use `MacOS.prefer_64_bit?` or `MacOS.snow_leopard?` depending on what you're trying to achieve.
Iterating twice ain't a big deal. It's a functional programming style.

We have pure function here (filterer) and a function with side effects (forEacher).

A bunch of pure functions are *very* easy for debugging.

Basically pure functions can be re-used and are easily composable. See [L65](https://github.com/brunch/brunch/blob/a65c57cfacce8f0c3dfb9aadd770b1459b99f5f5/src/fs_utils/generated_file.ls#L65) of brunch fork in LiveScript for example.
Тут, похоже, табы, а не пробелы.
Почему calendarExample?
Т.е. паттерн hellod сработает.
И надо правильную инициализацию теста сделать, без main()
That was probably my fault. I am not familiar with the technology.

On 09/12/2012 11:44 AM, Pete Muir wrote:
>
> In hotrod-endpoint/README.md:
>
> > +Target Product: JDG
> > +
> > +What is it?
> > +-----------
> > +
> > +Hot Rod is a binary TCP client-server protocol used in JBoss Data Grid. The Hot Rod protocol facilitates faster client and server interactions in comparison to other text based protocols and allows clients to make decisions about load balancing, failover and data location operations.
> > +
> > +This quickstart demonstrates how to connect remotely to JBoss Data Grid (JDG) to store, retrieve, and remove data from cache using the Hot Rod protocol. It is a simple Football Manager console application allows you to add and remove teams, add players to or remove players from teams, or print a list of the current teams and players using the Hot Rod based connector.
> > +
> > +
> > +System requirements
> > +-------------------
> > +
> > +All you need to build this project is Java 6.0 (Java SDK 1.6) or better, Maven 3.0 or better.
> > +
> > +The application this project produces is designed to be run on JBoss Datagrid Server 6.0
>
> Isn't it JBoss Data Grid? Why the word Server?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/infinispan/jdg-quickstart/pull/10/files#r1588510>.
>
this'll need to change when we land (and you've merged) chrome debugging. (bugzilla 740551).
> a = 'http://http://bob'
'http://http://bob'
> a.lastIndexOf('http://', 0)
0
> a.lastIndexOf('http://', 1)
0
> a.lastIndexOf('http://', -1)
0
> b = ' http://http://bob'
' http://http://bob'
> b.lastIndexOf('http://', 0)
-1
> b.lastIndexOf('http://', 1)
1
Please make sure not to add superfluous whitespace. It makes diffs harder to read.
Added a dest url parsing function here.
I suspect that at some time we'd want to move this to an include so we can use it in different places as well, modal dialogues for example.
Should test everything through sample_without_replacement.
Actually in my initial comment I had overlooked the fact that this exception is only raise for score functions that "needs_threshold" which is not the case for multiclass scoring metrics. I am ok with leaving the code as it is  and only introduce new markers when actually required.
cs_graph_components works with CSR data so it will produce another copy
type and shape missing
Seems to me that this is more or less copy-pasted twice. Am I missing something?
ok but Lasso fits with only 1 alpha so for now it's not useful.
should this be part of the configuration params? that you wont be required users to add a requirement to add the puppetmaster class?

looks good!
Wold you mind... we customarily put dst first, then src, then flags and outputs.  So:  fixNonFinite (dst, src, mode=NONFINITE_BLACK, pixelsFixed=NULL).
And if pixelsFixed is NULL, just don't store anything there.
I'm the "relevant person". :)  Engaged with the community maintainers, to be following up when I've more info,
I see. I've assumed the following already works which is why I've implemented `pointsTo..` function:

```D
struct Foo { static int field; }
enum F : Foo*
{
    a = cast(Foo*)null
}

void main() {
    auto f = F.field;  // NG
}
```

But it doesn't work, even with my fix. So the pointsTo function is unnecessary, I'll remove it.
Just initialize string. No need in malloc.
This parsing code is a good candidate to put in a function to avoid code duplication
La liste des sous-définitions disponibles pour le document sélectionné
yes it is but unfortunately there is no easy way to access them from groovy inside the project
Sure, easy enough.
See my comment in the PR. Lifting involves a dynamic invocation of `-setFrame:`, which behaves differently (wrongly).
>When you get close to the memory limit that would potentially allow you to not spill some buckets.

As opposed to only spilling some rows from each, which is what happens now via spillHashTable
The call to <code>useINotify</code> takes the <code>timeout</code> parameter passed to <code>monitorDirectory</code> and not the <code>checkInterval</code>  parameter for use in the select.  Once the timeout expires <code>(retVal == 0)</code> the loop breaks.  If there is no error, the loop will execute only once, either breaking out and returning NULL when <code>retVal == 0</code> or returning <code>cmp.getClear()</code> if a change is detected <code>(retVal > 0)</code>.  So the watch(es) is/are actually being added only 1x and then a select is called with the timeout.  The current implementation breaks the loop when the timeout expires, same thing with the inotify implementation, except without the looping. 

If understand correctly, breaking </code>useINotify</code> in to 2 methods would not change the behavior.
You don't need to call ensure again because it probably failed for lack of memory, so you're just going the extra mile to free some resources. And since it was going to fail anyway, there is no slowing down of any running code.
Given the second is only for backward compatibiltity of an undocumented feature, we could probably just remove it
Unread var.
I suggest using "protocols/uip/uip.h" for this define.
`(= (height rectangle) (width rectangle))` palauttaa booleanin sellaisenaan
Alright, but can they at least all be tabs or spaces? :P
what is the rationale for adding this force option? I would prefer if this was possible without force, or if force was a parameter.
Yes, of course you are right. I used Cython because originally I implemented the filter functions manually.
You decide ;-) What do you prefer?
i wonder if we can not provide some base class in RoutingExtraBundle that we extend here, or some helper class with static methods. this is essentially copy-paste from RoutingExtraBundle, right?
I'm not sure what you mean to say here.
Can we patch things to only lookup users who are marked as active? That seems like the best/simplest solution, at least conceptually. There's no real cognitive overheard, even if it means a bit more code.
Again ... this is a "standard" issue;  This basically checks that the images are in the right format and gives people a standard way to load them.  I agree it could be handled downstream ... in fact, we overload the equivalent method in the lfw stuff we're doing now ... I think the question is how complex we want to make the api for various possible non-standard options.  We could not only make the size variable, but also the mode (e.g. allow people to pass a desired mode and then do the conversion for them, and also various resizing option variables, &c...)   If you feeling is to give people maximum flexibility through this particular interface -- and set the "standards" via default values -- that's fine with me
If you think I should do it ASAP, I can.  It's probably not a lot of code, let me know.
For private functions, we might not need many of these checks if they are redundant.
@alexrp I don't understand why we should, but anyway added `detab`.
indentation is a bit off within the file I think
typo: should be "namespaces"
Since it's a collection, test data property should be TestData.Data.InvalidRoleNames (plural). Otherwise it looks like you're iterating over characters.
@sptramer I think you're wrong about the TIMOB-3432 request. Basically here is a scenario which affects Facebook module as well:
1. User action requires FB auth, so the module triggers openURL for FB app
2. FB app reopen the app back using redirect URL.
3. the app gets the redirect url during handleOpenURL call and stores new value in launchOptions
4. 'resume' event is sent which is handled by FB module to continue its tasks
5. Now, if the app goes in background and activated back, launchOptions still have url and FB module will be triggered again on 'resume' event
... _the_ \`resumed\` event.
firstRow may be undefined, theoretically
Switch to // for .dart copyrights
spelling in the line
" and will only compile (by default) for WebKit-based browsers (Safari, Chrome)"
Вообще говоря, нет. Ноопы надо.
??? что это вообще значит - uploadDrag?
Could be the following:

    return str_replace(array('webroot', '\\'), array('', '/'), "{$uploadDir}{$uploadDirMethod}");
isSpawnable?
hmm i havent used this myself yet. @beberlei contributed this.
but at first look your code should work.

the important bit is the `main` which is the name of the firewall in your `security` configuration
Don't do this in the class file.
Links still out of date :)
Sorry, I forgot to delete this line.
Yes I was inspired by your plugin to run the command. What you're saying is a good idea.
Spaces here.
This may be a deeper bug to investigate...
the check on the count should be separated to have a different exception message
Note that this line is unreachable since jarName cannot be null at this point.
Sure, but for this pull request I'll just make the lottery handle Mailman itself.  If we get an API in ClassSection for bulk registration then we can do it.  That might go along well with our upgrades for Django 1.4 which should allow us to use bulk_update.
double negative - maybe change this to `if dump_header and verbose` much easier to read.
I don't think so. `DllMain` isn't a place for such complicated tasks. That's why there is `SymInitialize`.
And it works fine on my PC.
Might I suggest the following? Seems cleaner to me.

Notice I moved the startTime inside the loop, to correct the problem in my previous comment.

````java

	private javax.jms.Message receiveCorrelatedReplyMessage(MessageConsumer messageConsumer,
			String correlationKey, String messageCorrelationIdToMatch, boolean correlationIdPropagated) throws JMSException {

		long timeLeft = this.receiveTimeout < 0 ? Long.MAX_VALUE : this.receiveTimeout;

		javax.jms.Message replyMessage = null;
		boolean foundMyReply = false;

		while (!foundMyReply && timeLeft > 0){
			long startTime = System.currentTimeMillis();

			replyMessage = (this.receiveTimeout >= 0) ? messageConsumer.receive(timeLeft)
					: messageConsumer.receive();

			if (replyMessage != null) {
				String jmsCorrelationId = null;
				if (correlationKey != null && !correlationKey.equals("JMSCorrelationID")){
					jmsCorrelationId = replyMessage.getStringProperty(correlationKey);
				}
				else {
					jmsCorrelationId = replyMessage.getJMSCorrelationID();
				}

				if (StringUtils.hasText(jmsCorrelationId) && StringUtils.hasText(messageCorrelationIdToMatch)){

					if (jmsCorrelationId.contains(messageCorrelationIdToMatch)){
						if (!correlationIdPropagated){
							replyMessage.setJMSCorrelationID(null);
						}
						foundMyReply = true;
					}
					else {
						// Essentially we are discarding the uncorrelated message here and moving on to
						// the next one since we can no longer communicate with its originating producer
						// since it has already timed out waiting for the reply. We are also honoring the original timeout
						if (this.logger.isDebugEnabled()){
							this.logger.debug("Discarded late arriving reply: " + replyMessage);
						}
						timeLeft -= (System.currentTimeMillis() - startTime);
					}
				}
				else {
					foundMyReply = true;
				}
			}
			else {
				timeLeft = 0;
			}
		}

		return replyMessage;
	}
````
Well, remember we are talking about MS which is by definition internal so I am not all that convinced about the use case to begin with since we have other implementations of MS and no one ever asked for the equivalent functionality.  So I am thinking about it as advanced configuration and at that point sure give me the Region instance.
Actually i say remove them. Those are definitely leftovers and after all if these are tests and if those sysouts cary any meaningful value they need to be refactored yo assertions.
If we are using a default delay, or a delay header that's just a number (not a release date), this logic will cause us to delay another full cycle. As Mark said in the JIRA, this is wrong (e.g. 10 minute delay, wait 9 minutes and restart server). We'll wait another 10 mins.

I think you need to store the wrapper and set an arrivalTime property in the wrapper; then you can use the arrivalTime and current time to determine the new delay needed for this message after restart.
Needs author tag - also, should be renamed to Service, TestService etc.

A service-activator is an endpoint that invokes a __service__; in SI terms, the invoked java class itself is not an 'activator'.


	<bean id="service" class="org.springframework.integration.jdbc.test.TestService"/>

	<int:service-activator input-channel="outputChannel" ref="service"/>
Composer rocks, now it's just a question of educating users ;)
DoctrineMongoODMModule has to be migrated too ;)
Jake, close, but not quite up to standards. A quick tuneup of indenting and we'll be good to go
Nevermind, I see.
Melhorar o nome.
Actually both. Also asking myself if those examles cases should be handled by one scenario or have one scenario each. Can you load those examples from some common file shared by some features/scenarios?
think we need to find a way to not duplicate this config() method everywhere in our objects
Please define this in the selection field itself because its used in only *one* place
Can you rename to ``sonata_top_bar_nav``
so, this is the filter you are passing to the function, right? that means that you want to get only the `_id`, right? I think, we can get, as you said, both things: the `_id` and the `URL`
Would it be worth generating a uuid (when this code is in use)?  Since this is client side, maybe something like http://www.pst.ifi.lmu.de/~rauschma/download/UUID.java
Yep, but I suppose that `the_m2ms` is a queryset with tastypie, they are using `.all()`
https://github.com/toastdriven/django-tastypie/blob/master/tastypie/fields.py#L739
We have a list.
Typo in `supoerte`=> `suporte`
Faut inclures le fichier mixins pour pouvoir l'utiliser par contre, je t'invite à aller le lire sur leur repository.

@import "twitter/bootstrap/mixins";
looks like a dd slipped in there.
I usually do that in a separate commit.
This should be set at the start of `downloadClientRecords`. In fact, anything download-related should be, mm?
ok I see. My update also changed the query for the Privacy Policy page, which also doesn't have a link_url attribute. Should that be changed back to query on the title?

Thanks for the walkthrough, still working my way through your code :)
|...\...
ИМХО, эту хрень надо перенести в `endUpload`.
Please place on its on line
I get linking errors when I remove bps.  Specifically, it complains about some pps functions and some navigator_invoke_* functions. 

There are no linking errors when I compile with -fPIC, but as before, gtest doesn't seem to work with that flag on.
Does the hash take into consideration files that were saved or uploaded with CR/LF format? If the file has different line endings than we expect the hash is going to be different. We should convert CR/LF to LF before doing the hash to make sure it is always the same.
It's mixing data types and making `related` seem like something it's not.  How would you support `obj.related[0]` to get the first related object of any type if you're overloading `__getitem__` to make it seem like it's not a QuerySet?

QuerySet-like objects have a defined behavior inside Django.  They have methods and attributes and if you access them like a dictionary they run queries and return a model (or set of methods if you slice).  Changing that adds mental friction, and if there's no technical reason for it, we shouldn't do it.