labofsahil/github-event-dataset-2012 huggingface dataset:
Sounds good. I forgot we already have the data, I thought it would require an extra HTTPS call. 
________________________________________
From: johnny.halife [reply@reply.github.com]
Sent: Wednesday, June 27, 2012 2:19 PM
To: Tomasz Janczuk
Subject: Re: [git-azure] Added support for 'env_variables' (raw using a CMD) & Windows Azure affinity group (#54)

> @@ -857,13 +866,20 @@ exports.action = function (cmd) {
>  <CreateStorageServiceInput xmlns="http://schemas.microsoft.com/windowsazure">\

I think we can simply show a console.log saying "Your current storage
account is located at XXX, while we're creating the Service on YYY. For
better results, performance, stability and as a best practice we recommend
you to use YYY for both"

That's what I'd change before merging

On Wed, Jun 27, 2012 at 6:15 PM, Tomasz Janczuk <
reply@reply.github.com
> wrote:

> > @@ -857,13 +866,20 @@ exports.action = function (cmd) {
> >  <CreateStorageServiceInput xmlns="
> http://schemas.microsoft.com/windowsazure">\
>
> If you are fine with a potential mismatch, let's just leave the code as
> is. It is probably not worth the extra complexity.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/tjanczuk/git-azure/pull/54/files#r1063820
>

---
Reply to this email directly or view it on GitHub:
https://github.com/tjanczuk/git-azure/pull/54/files#r1063841
So if you want to reuse a child view for whatever reason you need to explicitly remove before the parent is killed?
you can have a local bool that is false, which you set to true at the end of the try. Then you can add a finally that checks if it isn't true, and then sets  "isSynchronizedWithZooKeeper.set(false)". This scales much better with number of catches.
We're now trying to use some standard-ish colors for things (though a lot of older exercises still aren't totally consistent yet). The ones we use are defined in `css/khan-exercise.css` as `hint_blue`, `hint_green`, etc. They're also defined as constants in `utils/math.js` to make it easy to use them in javascript. For example, here you should be able to just replace `"blue"` with `BLUE` and you'll get the same shade of blue that we (try to) use everywhere else.

Also, FYI, there are also some MathJax macros defined for these colors too, so you can say `3 + 4\blue{x} = 5` if you want the `x` to be blue.
since this wasn't marked as a private API class it may break other peoples apps. This means we should increment the major release with this change
can you delete this commented line from your commit?
What is this crazy * notation about?
Sorry, copy-paste error and I wasn't testing with filtering turned on... this has been fixed, thanks!
use a setting and 12 as a fallback: https://github.com/fwenzel/django-sha2/blob/master/django_sha2/bcrypt_auth.py#L84
Wheesh; that is a mighty-big transform to have inside a loop. Have we done any performance testing of this on a thor?
ok. but you need to rename JCR_SQL to just SQL
you could do that in tags_helper.rb (you would have to create it)
Hmmm... yes that is easier, isn't it?
Why use require_relative instead of require if this was working before?
This is failing sporadically in our dev tests. Maybe we should xfail it until we figure out the fix. I tried working in it, but I can't reproduce the fail locally.
Why did you add this here?
You would overwrite commands with your rc values, regardless if you actually are in RC controlled mode and you have an RC signal.
I'd like to see tests for:

1. Private members that belong to groups are present and their profile data is hidden
2. Private groups that belong to groups are not present in the group memberships feed
3. Public groups that belong to groups are present in the group membership feed (if it doesn't already exist)
If in production, we should tell the browser to cache these requests for a very long time.
This is a 'gotcha' in JDI. By default it is set to [SUSPEND_ALL](http://docs.oracle.com/javase/6/docs/jdk/api/jpda/jdi/com/sun/jdi/request/EventRequest.html#setSuspendPolicy%28int%29). In a lot of cases it is critical to set the right suspend policy. In this case, there is just no need to suspend any thread when a thread is started or dies. I'll add a comment.
missing `get` method?
```js
'undefined' == typeof document
```
I'll clean it up
That's even better, as it reduces the check for IronPython to exactly one location (when setting the IRONPYTHON environment variable).
Could you explain a bit more. Is there something more than accessing the segment with 1 as the base?
Создайте @property для переменной DNA. Таким образом вы избавитель от необходимости конвертировать Cell в строку и сможете сравнивать элементы DNA для каждой клетки на прямую.
"В данном примере есть предрасположение для решения задачи в общем виде"
Задачу можно обобщить не только изменением длинны DNA, почему всех беспокоит именно это обобщение - я не понимаю. _Всё сделано до конца_ в соответствии с _текущим_ заданием. 
Когда/если задание будет обобщено - тогда пусть ученик и вносит соответствующие правки.

"если мы поменяем длину днк до 10.000 ?"
Тогда вы нарушите условие текущего задания.
A troubleshooting tip here is good.  But it might be better to fix connect.py to be better behaved.  (Apologies if this is the code I wrote that did this.)
At this point we're hiding the exception, which probably has all the good info to track it down.  So this might be more helpful to the user:

    except:
        print "Unexpected error:", sys.exc_info()[0]
        raise

You may even print some help text like: "Check that the protocol is correct" in there or something along those lines.
In the interest of having a common interface with ORM, I think we can skip the method for now. @asm89 is suggesting we port some of this stuff to Doctrine Common, so it's something we can certainly add there later on.
I had two motivations for choosing concat: over append:

 - Backwards compatibility for RACSubscribable. This obviously isn't crucial, given the other renaming I've done, but those changes have generally been to support more generic monad terminology. "append" doesn't fall into that category, because the generic operator would be "mplus."
 - I've only seen Cocoa use "append" terminology for strings and paths. Collections typically use custom phrasing, like -arrayByAddingObjectsFromArray:. Given how "append" is currently used in practice, it seems more appropriate to use a term that doesn't imply familiarity.
I guess protected would work for your cases as well. Or you're not extending form this class ? but anyway: I'm ok with public as well. :)
Should :RECLESSLY-P be T here?

Also, when googling I found a reference to UNIX:UNIX-EXIT here which does support exit codes. Should it be used instead? http://comments.gmane.org/gmane.lisp.cmucl.devel/10946

I haven't been able to install any version of CMUCL so I haven't been able to test this.
缩进貌似有问题，我这里是这样的：
```python
def sendmsg(self, msg, dest=None):
   if not self.joined:
     return
   logging.info('sending to irc: %s', msg)
   ch = self.config['channel']
   if dest is None:
       dest = '#' + str(ch)
   for m in msg.split('\n'):
     self.write(('PRIVMSG #%s :%s' % (dest, m)))
```
不管怎么样消息都是发出去的，只不过是dest是None的话那么就发送到channel里，如果不是那么私信发送到对应的 user
Yes, good idea.
Btw. the nitime exporter is another good reason for the RawFromMerge class
;-)

2012/10/11 Alexandre Gramfort <notifications@github.com>

> In examples/export/plot_raw_to_nitime.py:
>
> > +data_path = sample.data_path('..')
> > +raw_fname = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw.fif'
> > +
> > +###############################################################################
> > +# get raw data
> > +raw = Raw(raw_fname)
> > +
> > +# set picks
> > +picks = mne.fiff.pick_types(raw.info, meg=True, eeg=False, eog=False,
> > +                            stim=False, exclude=raw.info['bads'])
> > +
> > +# pick times
> > +start, stop = raw.time_to_index(100, 115)
> > +
> > +# export to nitime using a copy of the data
> > +raw_ts = raw.to_nitime(start=start, stop=stop, picks=picks, copy=True)
>
> maybe something like:
> http://nipy.sourceforge.net/nitime/examples/multi_taper_coh.html or
> http://nipy.sourceforge.net/nitime/examples/resting_state_fmri.htmlbetween magnetometers? all examples are at:
> http://nipy.sourceforge.net/nitime/examples/index.html
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/142/files#r1818122>.
>
>
is would write:

```
from scipy import signal
signal.resample(x_ ...)
```
Yes, that's absolutely fine with me! +1
Yes, that is expected. I'm not thrilled with the overall behavior, per my
earlier comment (though I probably am the one that defined it), but it's
expected. And refactoring the behavior should be part of a more
comprehensive ticket.

-Darius

On Sat, Oct 27, 2012 at 9:52 AM, lluismf <notifications@github.com> wrote:

> In
> api/src/main/java/org/openmrs/api/handler/ExistingOrNewVisitAssignmentHandler.java:
>
> > -							visitType = Context.getVisitService().getVisitType(Integer.parseInt(visitTypeIdOrUuid));
> > -						}
> > -						catch (NumberFormatException e) {
> > -							//ignore, could be a uuid
> > -						}
> > -						if (visitType == null)
> > -							visitType = Context.getVisitService().getVisitTypeByUuid(visitTypeIdOrUuid);
> > -						if (visitType != null) {
> > -							return visitType;
> > -						}
> > -					}
> > -				}
> > -			}
> > -			
> > -			//Reaching here means this encounter type is not in the user's mapping.
> > -			throw new APIException(
>
> You're right, I missed the flow throwing exception, but the original code
> just returns the first visit type if the GP is empty, is this the expected
> behaviour ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openmrs/openmrs-core/pull/96/files#r1959991>.
>
>
It's done by Hibernate by default.
Probably never...although we could probably pull the equivalent from the omnibus project using `Omnibus::BuildVersion. git_sha`
Unless there's a feed for each supported language at a separate URL I doubt it is a good idea to internationalize the feed's metadata.
@sorin-ionescu Well it explains where it comes from but still I'm not sure I see the point of the redundancy with vcs_info :/
Wouldn't it make sense to output this value?
I added them just to be consistent with the other methods. In case we add other qs parameters to the server, we dont need to update the client.
Not sure if there is a better way to do this. When matching the string on an HTML page capabara was looking for the HTML tags too. Used Nokokgiri to remove the HTML from the string.
ok. got it.
No, to make sense for the user, it's not the actual index of the Affine, but the 1st, 4th, and 5th affine that are found in the series (see _format_arg())
there is a space before $event. remove it :)
А делать программый продукт, который на каждый чих спрагивает что-то  - это
вызывать неебическое раздражение у пользователя
May want to look at http://code.google.com/p/java-bibtex/
Add an example in the class javadoc.
intentionally left empty?
I think I'd rahter be a little more defensive here, or is the file format so well-defined that missing items in such a mount list are unthinkable?
why check param->zoom here
You really don't need to force the disconnection. The connection can die a natural death.
Hmm... we already have this in the graph package.  Do we want two implementations?
I think that's an invalid data URI, isn't it?, so I'm not surprised it doesn't work...  It wouldn't work if you use any kind of invalid base64 data...
Shouldn't this value be ``--osapi_extension_path`` instead of ``--opai_extension_api``
Yep, I will have to move this file. Good catch.
Ah, didn't see that, let me adapt my code.
You're absolutely right, we should return here... I've updated the pull req.
Can this be simplified to return Builder instead of S?
Unless my eyes are deceiving me this looks absolutely fine.
This parenthetical would read better spelled out:

> if `withCheckout` is `YES`
Thank you for answering my question before I asked it :)
That should be a separate PR, which can involve introducing a new keyword ("asbytes"?) and deprecating the old one with a warning, while changing all uses within mpl to the new version.  Not urgent.

(bytes is not a python keyword but a relatively recent builtin type.)
Very picky, and I won't insist; but leaving the hash marks aligned really would look better in this case.
ouch! Can you mark this with a TODO to fix it in future? We should repair it with something like get all rows and then `rows[-1]`
Sadly, yes. I may re-run the TB11 tests before running the TB12 ones, just to see if I catch any add-ons. We didn't get any bug reports of add-ons marked compatible being broken though, so we might have gotten lucky. When I write these tests, they are literally based on every line of changes that occurred and quite often none of the stuff is used by any add-on at all in Thunderbirdland.
Merged, I'll let you update the bug with what you think needs to be done
still.


On Wed, Jun 6, 2012 at 5:07 PM, kmaglione <
reply@reply.github.com
> wrote:

> > @@ -11,7 +11,8 @@ def detect_type(err, install_rdf=None,
> xpi_package=None):
> >      translated_types = {"2": PACKAGE_EXTENSION,
> >                          "4": PACKAGE_THEME,
> >                          "8": PACKAGE_LANGPACK,
> > -                        "32": PACKAGE_MULTI}
> > +                        "32": PACKAGE_MULTI,
> > +                        "64": PACKAGE_DICTIONARY}
>
> On Wed, Jun 06, 2012 at 01:34:49PM -0700, Matt Basta wrote:
> >Does this cover the whole bug? I.e.: is there more to do in terms of
> validating the structure/content of the new dictionary format?
>
> The rest of the format is pretty much the same. I guess it would
> be good to make sure that there's a dictionary directory too,
> but it's more important right now to just make sure they can be
> uploaded.
>
> --
> Kris Maglione
>
> The tragedy of modern war is not so much that young men die but that
> they die fighting each other, instead of their real enemies back home
> in the capitals.
>        --Edward Abbey
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mozilla/amo-validator/pull/157/files#r939777
>
Nit: Extra whitespace below

Test looks good!

There's a couple entry points to get to the wallpaper settings I noticed; we should reference the approach we're using in this one.
You're certainly right regarding the variety of similar distros. I'll push a change to put the changes into the yum::yum recipe later.

Regarding the copyright stuff, I have to ask my company if they are okay with signing the CLA. Thanks for the pointer to the explanation!
You are missing a comma at the end of this line
There's `#iso8601` for that: `@versions.first.updated_at.iso8601`
Any reason why you use .selector here?
This domReady.js file is referenced in a lot of files in `tests/qunit/tests/unit/` and `tests/qunit/tests/integration/`. We should remove all of those references or leave the domReady.js file in the system.
yes that's the multiclass version and in the prev version (0.95) it was 
y2 = (uint32_t)data_y[i]-1;

as well while in shogun it was y2 = (uint32_t)data_y[i]; so i guess that's a difference between the libocas and shogun, or?
this is definitely wrong - when maxtime = 0 it will not do a single iteration
I'm not sure about the wording of this, as it implies upgrading (i.e. changing from deprecated functionality).

Isn't this documentation supposed to be reference for SiteConfig? Wouldn't this be better suited in the upgrading guide instead?
Should use `File::get_file_extension()` (which in turn uses `pathinfo()`). 
Also, both the old and new implementation have a problem with double barrelled extensions like `foo.tar.gz`.
I'm not sure if there's a standard declaration to detect them (e.g. distinguish it from `foo.bar.gz`).
But either way, the MIME type list should probably contain "gz" as a key, otherwise a quite common file format will go undetected here.
wrong indent - should 4 spaces.
Nemohlo by se to jmenovat timer_interrupt_handler? Protože i jednotlivé timery mají svůj callback, ale tohle je obsluha přerušení. trochu mě to zmátlo.
I tried to avoid this, so there was only one place where the schema lock is needed and may or may not be held (when opening a handle for the first time).

Where is this needed now?  In other words, what is the protocol for accessing the schema lock?
Because that's the name of the tag in Symfony and packagist. But I guess there's some magic why it works without the `v` then.
This is why it is important that the next filter is also a filter, so that we can traverse it.
I HAD already removed it. My memory isn't faulty after all. It must be my
git skills that need polishing. I'll try and get it committed.

On 10 December 2012 08:19, Jocelyn Fiat <notifications@github.com> wrote:

> In library/server/wsf/router/wsf_router.e:
>
> > @@ -53,6 +53,9 @@ feature -- Mapping
> >  	map_with_request_methods (a_mapping: WSF_ROUTER_MAPPING; rqst_methods: detachable WSF_ROUTER_METHODS)
> >  			-- Map `a_mapping' for request methods `rqst_methods'
> >  		do
> > +			if attached rqst_methods as l_rm and then l_rm.has ({HTTP_REQUEST_METHODS}.method_get) then
> > +				l_rm.enable_head
>
> I guess the change in map_with_request_methods is not relevant anymore.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/EiffelWebFramework/EWF/pull/16/files#r2361207>.
>
>
Friend? Likeneighbors?
I don't remember if we support the option, but if we do, we should add the --xunit-file option to this to ensure its always hiro.xml
OPTIONAL: I think this is more readable, and it's also more functional:

```ruby
default_params = { :api_key => PistonApi.api_key }
PistonApi.sandbox? ? default_params.merge(:sandbox => 1) : default_params
```
Same as the above but to inform the user that internet is good but the friendcash web server is not responding.
But I'm not currently using it so you can skip these :)
Don't do requires in your controllers!
What is this map??? `lambda s: s` does nothing?
Vau, I didn't know that `later` is different than `latter`.
I don't understand the question?

But yes, you should probably have some flag to store whether we believe user is logged in or not. And you update that flag in the task. And on change (atomic!) you update everybody of this change.
This, `column_map`, and `insert_columns` could probably be class attributes.
This (and the one below it) need to specify the full path to the view:
```python
handler404 = 'fullhouse.dashboard.views.handler404'
```
It seems that `build_customer` is a concern that's external to ActiveMerchant - any issue with it being stripped out?
style.  Lining up the `= xml_fixture(...)` with the rest of the RHS's would be nifty.
if (Creature* Yarzill = me->FindNearestCreature(C_YARZILL, 50.0f)
why not reply something understandable?
Ive started using .update as well. Right after this feature :grin:
I also think of AbstractUserProvider which would build its functionality on top of this identity class and its relation to user class. 

It is up to you use it or create your way of storing idenities and making relation with a user.
there is an identity class which is mapped super class (like FOSUserBundle User class). a developer can use it in his UserProvider implementation.
Another note in reply.
> Insert here usual bracket-related m-c nit.

https://mxr.mozilla.org/mozilla-central/search?string=[^\%3F]%2B\%3F[^\%3A]%2B\%3A&regexp=1&find=.jsm&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central
I'm not aware of that rule.
Would it then "snap" to actual events?
I see 1.225 appearing in various places; it should probably be a named constant in a header somewhere.
No problem, thanks for looking into it in detail! My reason is that plone.scale uses a different default, 88: https://github.com/plone/plone.scale/blob/master/plone/scale/scale.py
No here I force the preferences to be written out to file, for the command to execute correctly. See next comment.
Aha, yeah I think you're right. This'll fix the problems for this pull. Hopefully, the actual changes I've made will fix the problem… in general :)

See the extra commit in this pull that fixes the couple of problems
List-wise deletion. This was superfluous since statsmodels takes care of it. I removed it. Thanks a lot for the other comments: 	fd8031e
TODO once discussion below is resolved
I just don't like those local variables that are created only to be passed somewhere else in the next moment. But you're probably right, it might be a bit less readable that way. I'll remove that if you decide to merge the other part of this PR.
Also, be sure to add a new line at the bottom of files :p.
Hm, just to be consistent? Because code reader then wonders, why is here something different ...
Can we move all these little functions into a closure or bidi class.
Because the rounding is done to any number, perhaps a better suited name for ```timeAccuracy``` is ```numberAccuracy```
I tried that and didn't work. actually instead of TNArchipelLibvirtEntity.on_xmpp_loop_tick the TNArchipelEntity.on_xmpp_loop_tick will get executed with is an empty method. but I moved libvirt_failure method to TNArchipelLibvirtEntity.
Nothing, I deleted the params I was passing and just forgot to kill the {}.
Same here.  Should have some kind of comment about how this file was created.
really not sure how this got removed.
You should be using count() instead of just counting manually.
If this ```<ul>``` gets class unstyled, like this: ```<ul class="unstyled">```, then the bullets that appear with your Attempt summary when you click on the chevron go away.
I see -- get all the video downloads for this student, then decide which ones are used for which ID's in the inner loop (below).  Good.
close is a no-op for ByteArrayOutputStream, but no strong feelings about it.
feels a shame to throw away all these tests?  are there replacements?
I know too. But dmd 1.072 said complain.
Do a global search for `boolean`.
this is a small "L" like "Layout"
The avarage of two numbers a and b is: (a+b)/2

This doesn't pass the tests
"is already defined like this" -> "was written like this originally!".
Sweet thanks
What is there produces the output I would expect. https://github.com/resolve/refinerycms/blob/master/core/lib/refinery/core.rb#L96 is putting it into a string, so the Float gets translated properly.
Since I can grab it from an index, I prefer to. That way I don't have any coupling in the code across endpoints (so if I break config, then I don't break indexes in the process).
I've changed the place of the routes.

As for the way I'm removing them from the ```routing_dev.yml``` file... well if one should change the order of the routes for the AcmeDemoBundle it should be in the new dedicated file rather that in the global file imho.
Why do you need this? startPlayingFromStart() and these functions are used to start playing a playlist from the  'first available track' when double-clicking on a playlist in the sidebar.

If you call startPlayingFromStart() and it activates but does not start playing an item (in this case if m_readOnly is false or m_manualProgression is false) that sounds really counter-intuitive.
Documentation could be better here.  Is street_number a String or a Fixnum?  How about zip_code?  lat? lon?  Are all of these required?  What happens if zip code is null?  Will I still get a result?  Will it fail silently, or return some kind of error, or throw an exception?
you also can see which file could not be loaded or has been loaded
Use less line, chop it!
oh cool, will do :)
Aici nu ar trebui un <code>return false;</code> ?
Two small questions here:
1. I what does passing the index on the chunk method do since you can only configure one chunk per tasklet?
2. I would have expected the following give the requirement to use the <tasklet> tag in the XML (while I prefer the syntax you provide here...for consistency the XML requires the tasklet declaration...).

    steps.get("step").tasklet().<Trade, Object> chunk().reader...
Minor: we can wrap this in an else-block. If the value was undefined, it's now just an empty string, so we don't need to try and escape it.
Вместо типизированных reader'ов можно сделать общую функцию. Они все реализуют интерфейс Closeable.
String.valueOf лишний
??-?????, ?? ??????? ? ???? shell, ?? ? ??????? ??? ?? ????????.
\s - это разделители
We seem to be missing the JBoss AS instructions? I thought we proposed putting the Tomcat instructions in a README-tomcat.md?
Well... crap.  This may get way more complicated then.
may については、「終了する可能性があります。」という訳だと違和感ある？
Here is an example. You need to use _('label_schema_settings', default=u'Settings') so that it will show Settings rather than the msgid if there is no translation. And existing translations that specify a translation for Settings need to be updated so that it is a translation of label_schema_settings instead.
int_max, int_min and is_in_mask could be shared by core8 and 16 - or is there a difference I didn't notice?
Just moved it out of ``preprocessing.py`` into ``base.py``.

```
ebb42e78 sklearn/preprocessing/preprocessing.py (Mathieu Blondel 2012-01-21 05:29:42 +0900 593) def _is_label_indicator_matrix(y):
ebb42e78 sklearn/preprocessing/preprocessing.py (Mathieu Blondel 2012-01-21 05:29:42 +0900 594)     return hasattr(y, "shape") and len(y.shape) == 2
ebb42e78 sklearn/preprocessing/preprocessing.py (Mathieu Blondel 2012-01-21 05:29:42 +0900 595) 
ebb42e78 sklearn/preprocessing/preprocessing.py (Mathieu Blondel 2012-01-21 05:29:42 +0900 596) 
ebb42e78 sklearn/preprocessing/preprocessing.py (Mathieu Blondel 2012-01-21 05:29:42 +0900 597) def _is_multilabel(y):
7b060267 sklearn/preprocessing.py               (Lars Buitinck   2012-05-18 09:25:21 +0200 598)     # the explicit check for ndarray is for forward compatibility; future
7b060267 sklearn/preprocessing.py               (Lars Buitinck   2012-05-18 09:25:21 +0200 599)     # versions of Numpy might want to register ndarray as a Sequence
6d041623 sklearn/preprocessing.py               (Lars Buitinck   2012-05-18 12:28:30 +0200 600)     return not isinstance(y[0], np.ndarray) and isinstance(y[0], Sequence) \
855257f6 sklearn/preprocessing.py               (Lars Buitinck   2012-05-24 16:08:07 +0200 601)        and not isinstance(y[0], basestring) \
7b060267 sklearn/preprocessing.py               (Lars Buitinck   2012-05-18 09:25:21 +0200 602)         or _is_label_indicator_matrix(y)
```
Can you suggest a better formulation? Btw the user should never be able to produce this value error. You have to change the sklearn implementation to actually get here.
It would be more useful if the warning was with respect to the actual labels, rather than the binary labels obtained from `LabelBinarizer`.
thanks @GaelVaroquaux and @ogrisel - this discussion thread started with gael and me discussing the current differences between the method implemented in this PR and the spectral embedding, to understand better whether it would be more useful to add options to spectral embedding (like kernelPCA has options for various kernels) or to write a separate estimator as was in this PR originally (btw, the PR was only partly done to get a sense of the differences with existing methods). 

it went off in a *slightly* different tangent, but this has been very helpful. someday over a beer we will have discussions on how large code bases can be maintained manageably. 

going forward, i'll finish coding the rest because it's useful to me even in a branch and i'll put references to practical impact.
I would rather use more standard refences such as elements of statistical learning and wikipedia.
The link to the thread seems pretty out of place. Also, I guess alternative names should be mentioned. This is the multinomial logistic regression loss, right? aka softmax loss aka max entropy?
I think we can rely on later identifier lexing, so this is sufficient .
owner shouldn't be allowed to be NULL
Please put a comment in the Jira
maybe we can replace them with an Enum for type safety.
Yeah, when I understand the algorithm for which AKI goes with which region, I'll update the `create_micro_bosh_yml` script. That will mean we don't have to document it. That script will make this tutorial a lot easier to follow hopefully!
> Maybe worth adding the commands you are using to https://community.jboss.org/wiki/ReleasingHibernateSearch so that we can learn as well ;-)

impossible, I did that 6 months ago ;)
I don't think alloca is appropriate here. Size is not bounded, and speed is not critical. Can you use a PointerArray?
These key spills, contain the initial dedup row key only.
It's a good catch, lonnen pointed it out (as a top-level comment).
Yes, it's configured just like this on our servers and just works.

The options for nova-novncproxy are different from nova options, they can be found in the code at : https://github.com/cloudbuilders/noVNC/blob/master/utils/nova-novncproxy#L38-68

And as said in previous comment, it's the compute node which “creates” the url for get-vnc-console.
Yes, more like /[a-z]{2}(-[a-z]{2}){0,1}/ (matches en, de, en-us, ch-fr, etc)
Actually, the correct change is to change the class name in library/Zend/Json/Server/Smd.php to be "Smd", not "SMD".
Should use `this.secure`.
Actually, the default value for this technique is "none", meaning nothing as to be done. Hence the none with a lowercase
Why not follow suit, and do mongo_admin.py?
Commente cette partie du design, pour l'instant on n'a pas de contenu associé.
It's the input to the state machine, not input to the function. The point is moot anyway, this code's being deleted as soon as my test run finished.
"Creates a <code>ServiceTimeoutException</code> with specified message. "
same here. @stof any comments on this?
I think you're looking for:

```ruby
course_membership.completed_tasks.find_or_create_by_task_id(:task_id => task_id, :description => status)
```
wrong text.
The web browser plugin for Ubisoft Uplay PC before 2.0.4 allows....
You aren't using that attribute for anything in your old code. There is no regression here. All you did was print it to the screen.
Why do it this way? The Railgun method gives you all domains the machine knows about, whereas this only gives you one domain store din this regkey. Presumably the currently joined Domain.
I would like to check res.code before res.body. I can add it by myself when merging. No prob
Right. Forgot to change that.
Sorry height and width always confuse me. I feel ``image[y, x]`` looks weird. But I can rename them if it makes it easier to read.
Hm, won't the way you did this make it only work if using Rails 3.2?

How are you testing that this still works with Rails 3.0?
all these use statements are useless as the classes are in the same namespace
Updated author and version javadoc on my new classes.
CZML should be Czml when using in any source of JavaScript name, for example, see `processCzml` and and of the CZML types (`CzmlBoolean`, etc..)
Looking at the code some more, while I agree that something like `getMatrixElement(matrix, column, row)` should be in all of our general Matrix classes, Quaternion.fromRotationMatrix is better off indexing directly into the Matrix array directly.  It avoids the extra function call/copying/error checking that the real methods would have.
Well, if we are going to correct people for not following "best practices" that kind of makes them "rules" doesn't it? ;)

I'll add that in the future, if I accidentally make a style change to code you wrote, just change it back as part of the merge without even telling me.  It's probably easier for everyone and avoids these types of debates.
I personally like it when sample code shows all of the options I can set.  It lets me know what I can easily change.  We could also comment out the options but leave them there with a note that they are the default values, but settable.
Will this break a lot of users?  If so, we should provide an example or two of the most common fixes.
`obj` doesn't make sense here.  Maybe instead `key`.
WebView, VideoPlayer, MapView + most of our widgets. They all add subviews which are not TiUIViews
iOS supports it.
Feel free to submit another.  I just went through the Platform team.

On Mon, Jul 30, 2012 at 10:38 PM, Arthur Evans <
reply@reply.github.com
> wrote:

> >  - Bryan Hughes
> > +- Ping Wang
> >
>
> Arthur Evans ;-).
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/appcelerator/titanium_mobile/pull/2655/files#r1267418
>
The difference in indentation here leaves me to believe there may be an indentation error.
Probably we should change the code like this:

1. negate the `if`-condition
2. switch the `return` statements (`return null` is now on line 111)
3. remove the `null` from `return null` on line 111
4. add a newline before the actual return.

The code is much more readable if the "success"-return statement is placed at the bottom.
Remove the EAP bit.
I agree, although there is no way to disable this check in jslint and I don't want to hack it and end up maintaining it ourselves.

Basically it's a trade-off I'm willing to make.
The language does not consider anything other than static arrays to be static arrays and certainly doesn't consider an `int` to be a static array of length `0`. You're attempting to introduce a new concept rather than simply using introspection on types, and I expect that you're going to have a very hard time convincing the Phobos devs that it's a good idea.
Thanks. #903
> And as I said, it makes about as much sense to say that a multimensional static array isn't a static array as it does to say that a yellow car isn't a car.

And in my opinion you are telling: a car parking without cars isn't a car parking.

> Clearly, we're not going to agree here, and this discussion is going nowhere.

Of course! But thanks for your time.
The only use for forwarding is "forwarding". You are welcome to propose a better syntax.
viewModel?
oh you better pray you never do that :-)
Rather than use MyBean, why not call this MessageProvider
Why? That whole file is only used for lookups, the file itself is never included, let alone that the class is actually called.
To be a bit more readable, move the SQL statement out into its own variable and put the `?,` stuff on their own lines.
Both ranges are invalid.
The value are empty here to show users what variables exist. You can copy the file over to "phpunit.xml" and "phpunit.functional.xml" and then fill out the parameters with your values. That is much more explicit than having ENV Variables (in my opinion)
they are useless here. The DI extension class will always override them with the value coming form the semantic configuration as a default value is defined in the Configuration class (which is the right place to put the defaults of the semantic config as the doc tool will be able to use the default value too)
Hmm, I grabbed that data from Alex's post here: https://groups.google.com/d/msg/openaustralia-dev/LcELLUVilsE/Dcjh7ItPZr4J

I'd assumed the data he used included the new WA boundaries because it's marked as updated in 2011. I'll take a squiz at the ABS SED shapefile.
Can you remove this empty line as well?
лишние пробелы,
внизу тоже
Только если само тип не может вывести.
thanks for the discussion. fixed this.
We probably want to just use `Chef::Knife::UI` here which provides our abstraction above STDOUT/STDIN and by proxy highline.  Try:

    require 'chef/knife/core/ui'
Can this be `body.i18nable > div` to make this more explicit?
why oh why an empty line in a two-liner?
I'll remove this line during the merge
You're right, Gary! Thanks!
I've tested it this night and it is realy root context is refreshed first :-).
I add this `if` to avoid getting several ContextRefreshedEvents from parent-child context (in most cases it happens in the Spring-MVC applications) .
However to add some 'startup indicator' (see comment on `reschedulePersistedMessagesOnStartup()`) will be enough to start `DelayHandler` only on one `ContextRefreshedEvent`. And in this case `implements ApplicationContextAware` will be redundant
I suppose sendingTaskExecutor could simply be NULL by default, since the doStart() method will always create when it is null. The main advantage would be IF we ever decide to change something about that default instance, it would only be in one place (instead of risking the change only occurring in one of the 2 places).
Required by TOS to use the API
delete the index document and recreate
This is Perl 4 syntax. Today you would use `strip($nextline)`.
I get this now -- ignore me :).
This gem directly writes to that column using `update_attributes`, so by default it makes it writable.
the space after and before args is not consistent, rather no space is better imo
Don't use ascii encoding - JSON is usually UTF8.  You can avoid the issue entirely by just taking a TextReader.
Please add the needed ``=`` to match the length of the title
change to &&
well I guess we can remove these and add a poll.. I suppose later..
Please import from feature absolute import, just to support older Python versions, if some crazy person will try to run things there.
Wording suggestion: `helptext="What kind of quote to display while loading a game"`
personally I think one a one liner if statement is not good. I would write:
``healingInterval = (difficulty != null && difficulty.difficultyID == 3) ? 15 : 25;``
and remove line 56
The problem is that the controller can't construct a schedulermessage object since it can't have a handle to the SchedulerProtcolet. Should we just allow for ID or SchedulerProtcolet constructors?
Aack..I thought I had a commit with comments cleaned up. Wonder whether it was not included in this pull request.

I'll do a separate pull request with the updated commits.
Whoops...now I'm not sure we're talking about the same thing? These three calls to `stringByReplacingOccurrencesOfString:withString` are only meant to catch common typos in queries; nothing to do with validating field lists.
This is what I'm saying, you don't need to resolve any intermediate directory to make up a new path, which you'll probably recursively create it anyway.

It either exists or not, and there's nothing this check can provide that you need to create it afterwards. I believe the createDir function (whatever it's called) actually does create recursively, so this check is not necessary for the old behaviour.

My opinion is that, if it fails to resolve the first, and it's not absolute, append it to current dir and return.
Seems like there's a word missing here. The API level isn't running on the system, the OS is.
"Android API level of the device's OS."  OR "Android API level supported by this device," or something of that nature.
Should read `all properties set on the proxy` for clarity.
This should be implemented in the sql or removed.
Although, it's good to be able to tell what you are linking to.
Most jars are versioned.

It's only in .classpath, that we need to make the change. So it's not that much work.
I think we should use of a sekizai tag here to provide the css resources so that the style is available in any cms page.
Quick comment on what you are splicing out (i.e. locale name)?
We *may* want to change this to the new overlaywebview.initialized? I don't event know where the ui.init event comes from.
event names are wrong - 

http://github.rim.net/webworks/webplatform/blob/master/framework/lib/chrome/events/application.js#L123
rename to m_lock
elif?
You could have used `divides?` here instead of `mod`