labofsahil/github-event-dataset-2012 huggingface dataset:
Hello David,
I reviewed CHOOSEN function and compared it to RndSampleWithoutReplace,
CHOOSEN returns the first N number of RECORDS starting at position STARTPOS from any RECORDSET
while RSWR returns a pctSize percentage of RECORDS from a DATASET(Types.DiscreteField), 
however  RSWR does a random selection of "id" based on a temporary rnd field, 
returns one RECORDSET (genIdList) of Types.DiscreteField with id:= new Id and value:= original id, just in case need to track back
and another RECORDSET (genSubSample) that contains all RECORDS from original Dataset having id in genIdList.value, 
these new RECORDS are generated with id:= genIdList.Id, number:= originalDS.number and value:=  originalDS.number.
The purpose of this method, as well as other methods in this module, is to generate a subsample of the original dataset but treating records with same id as a unit.
This whole nested `if` can be a lot flatter, I think.
I added that because the agent recipe
https://github.com/lusis/chef-logstash/blob/master/recipes/agent.rb#L93
references
shipper.conf not logstash.conf

On Thu, Jul 12, 2012 at 4:01 PM, John E. Vincent <
reply@reply.github.com
> wrote:

> > @@ -6,7 +6,7 @@ exec 2>&1
> >  LOGSTASH_HOME="<%= node[:logstash][:basedir] %>/agent"
> >  GC_OPTS="-XX:+UseParallelOldGC"
> >  JAVA_OPTS="-server -Xms<%= node[:logstash][:agent][:xms] %> -Xmx<%=
> node[:logstash][:agent][:xmx] %> -Djava.io.tmpdir=$LOGSTASH_HOME/tmp/"
> > -LOGSTASH_OPTS="agent -f $LOGSTASH_HOME/etc/logstash.conf -l
> $LOGSTASH_HOME/log/logstash.log"
> > +LOGSTASH_OPTS="agent -f $LOGSTASH_HOME/etc/shipper.conf -l
> $LOGSTASH_HOME/log/logstash.log"
>
> @bryanwb can you undo this change (or I can). This will break existing
> installs and force a restart.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lusis/chef-logstash/pull/14/files#r1149728
>
I feel like it should be like this:

```javascript

// default method is addition
        method : function(data) {
          data = _.reject(data, function(d) {
            if ( d === null ) { return true; }
            if ( _.isUndefined(d) ) { return true; }
            if ( _.isNaN(d) ) { return true; }
            return false;
          });
          return options.method.apply(this, [data]) || _.sum(data)
        },
```
aiiiiiiiiiiiiiiiiiii
ok, in reverse order:

i did not put it in the header object because the header object does not know what page it is a part of (at least that i know of, i'm not a python expert yet). if i did not put it in the page object rather than the header, then i would have to duplicate both the header object and the search_for method in each of the affected pages. replacing the else with an if for Home and SearchList would actually break the themes and extensions tests =).

the reason the else is an else rather than another if is because i checked all of the lists. searching from the homepage, extensions page, themes page, mobile page, and dictionaries page (the last two of which we do not test) and all of them use the same list format, with the same locators.

i could do it the way dave says...but it would require putting the 3 line search_for code (with different return types) into the Personas, PersonasSearchResultList, Collections, and CollectionSearchResultList classes.
عجب خطی
You are correct.  I think it's a debugging artifact that has been corrected and can go away.
`Refinery::Plugins.active` depends on the user being logged in — if you're just trying it out in console, it won't work.
Having dummy app for each extension would make sense when these extensions are extracted from main app. Given the fact the extensions are generated inside working rails app I don't think we should add another rails app just for testing.
If so, please toss in a unit test for this specific case along w/ the fix.
This should have the `{}` as well.
Access checked not, renamed function.
Stuart documented the error coming back and I removed the config that was sent back in the callback.
We always order our requires in the following way:

1) 3rd party requires
2) OAE requires from other modules
3) Requires from the current module
> It's not an expected outcome, that should really not happen. 

I know. And that's why you are using an assertion. I just think that an exception would be better in this case. And, I checked the doc for ``IllegalStateException``, and figured it is not the right exception to throw in this case. I believe ``UnsupportedOperationException`` is the one.

> I don't think we elide assertions even in compiler release builds.

What is true today may very well change tomorrow, and the less thing you need to remember the better.

> I am not sure what to do: throwing an exception would not convey the underlying assumption of the code. On the other hand, maybe the calling code wants to be able to recover... hard choice.

What do you think of ``UnsupportedOperationException``?
Convention is to add empty-parens for methods that have side-effects (and have a feeling that ``ConnectionPort.next`` does have side-effects :))
I see a method that converts String<->ItemStack, is that what you refer to? (ParseItemNotation)
поверни відступи. не буде сміття в diff'і
There's no way to cancel in this loop, so unless we get a JS error that breaks the loop I don't think that can happen...
в отсутствие комментариев мне лень разбираться в этом нагромождении кода. он правда работает?
не уверен что нужно приведение типа. правда нужно?
@wiifit к вам никаких претензий. Мы с @mvmn мило побеседовали на вашей площадке, спасибо!
http://ru.wikipedia.org/wiki/%D0%9C%D0%B0%D0%B3%D0%B8%D1%87%D0%B5%D1%81%D0%BA%D0%BE%D0%B5_%D1%87%D0%B8%D1%81%D0%BB%D0%BE_(%D0%BF%D1%80%D0%BE%D0%B3%D1%80%D0%B0%D0%BC%D0%BC%D0%B8%D1%80%D0%BE%D0%B2%D0%B0%D0%BD%D0%B8%D0%B5)
Ну вообще, да, разница у них есть, но проявится она только когда значения DNA смогут принимать не-константные значения. 

Если, например, использовать какой-то набор константных значений, этот вариант сойдет. 

--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev
quite long line, coffeelint fucks things up when stuff is >80 symbols. mind splitting this please?
Right, per user limit will not be an issue for most web sites. However, it is not required by [authentication](https://dev.twitter.com/docs/auth/implementing-sign-twitter). POST oauth/access_token is the last step in the workflow providing user_id and screen_name.
This PR is breaking a feature (sms notification). I understand you concerns but I cannot land that as part of my PR.
It'll still be `assign` technically, but I'd prefer to just leave off any memory management attribute, and document its semantics instead.
*`+currentScheduler`
I think this test should be combined with the previous one, since they're effectively testing the same thing.
Exactly. My latest commit reverts back to the previous code, but retains the check for record.frozen? which was included to prevent problems when destroying records with custom filenames. All tests pass now so we should be good to merge.
stc stands for source time courses. I seem label time courses (ltc) could be our naming convention to designate label means (more generic that the just the mean) so maybe mne.extract_label_time_courses or mne.extract_ltc_from_stc ... not convinced myself...
#181 fixes it, please merge ASAP. 

@Eric89GXL we should take failing tests seriously and not just update the .rst file. Also, before merging a PR always try to run the tests and doc-test, or make sure the person sending the PR confirms that it doesn't break anything.
for the sake of terminal friendliness break line at the column the title ends.
Except for getById, yes.
I stopped using this, so it can be removed.
The git one the main concern is `git://.*` or `.*\.git`
From what I can read, Apple will not send anything back when the push message is accepted. Thus, we will block here indefinitely. Seems consistent with what I'm seeing as well.
I can't find it in the current standards (it is a bit difficult to search because it is separated in 4 files), but the main rule is:

  > Imitate the existing Symfony2 code.

And we can find this all over the S2 code. For instance [here](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Routing/RouteCompiler.php#L89) and [here](https://github.com/symfony/symfony/blob/master/src/Symfony/Bundle/FrameworkBundle/HttpKernel.php#L220).
意思是最少访问多少次才会保留下缓存中
and
和
@penberg silvertip.MessageParser requires a type parameter.
`const Entity&` же.
What are you line-endings currently set to in sublime?
hasSurveyPermission should be checked earlier in the function to prevent unnecessary queries
using the long form of inject works just fine, but you could use the simple version: 
```ruby
array.inject(:*)
```
This would probably have been better in a JSON helper class than specific to source version
Do we have the URLs to the packages above? If yes, please include them.
@arnaud-lb while moving the data to yaml I found phpunit complain to this string `<span class="fselectmenu-label"> </span>` the diff was in ` ` which was actually: `&#xA0;` . Could you check whether it is what it should be.
It wasn't my intention, it's just a habit :P You can lock the version, of course.
Drop this line. Kinda inferred from the above.
Can you not add this whitespace?
Can maybe chose a better name...
Are we sure the flavor id gets updated only after a successful resize has been completed ?
NGHUTPR but if you like you might try out using the tree() construction I showed off earlier
So they'd be two different files - would we still combine them into this config object? Maybe the "load_paste_app" should just avoid changing the values in Config.instance, or actually be moved into a separate class.
Looks like the cookbook uses strings.  Corrected the README to be standard.

emopop:graphite(interface)% grep -R 'node\[\:' .
./README.md:* Ubuntu 11.10 (Oneiric) - change node[:graphite][:python_version] to "2.7"
./README.md:* `node[:graphite][:password]` - Sets the default password for the graphite
getNumOwners/getNumSegments - these are cache-global config params, any point in having them here as well? Perhaps for convenience?
If it's per type, it should be keyed on Class, not String, and it should be a weak key hash map to avoid leaking classloaders.

This map should also be cleared when the cache is stopped.

Finally, I can see what you're trying to do with an AtomicLong value, but the first time the type is added, you have no concurrent guarantees with this map. A concurrent map would be IMO, better suited, and you'd avoid the need of AtomicLong and you could have just a Long.

So, I think you need a concurrent weak key hash map here, and we have one of those in `org.infinispan.util.concurrent`
I have no idea what I was thinking.. too long ago :D You're both right. I think it was a leftover from some previous draft.
you can move this bit to the top of the page with the other imports then it can be used within the class.
аргументы не плохо бы привести к одному знаменателю
> Тебе не кажется что это приведет к неправильному использованию этого класса ?

нет. не могу себе представить.
> можешь привести пару примеров

список ссылок в поиске. содержит параметр и значение.
I don't understand this change. Is it at all related to the issue? And with the additional `triggerEvent` call won't `moveend` be called twice here?
BTW I realize that you chose isMiss because most checks would be testing for a fail before regenerating the cached value, but I still think it looks funny in the interface.
Ill have a look in this once it is merged so that I can browse around in the code
I believe you may use it for any type of features - only distance is required, right?
If we want to include the thing with the parameter m_q for cover tree we
should compute the distances from the query point to the neighbors 
found since JL's cover tree doesn't give them ordered by distance.

Does it pay off? My idea would be to do this if m_q != 1.
I'd say remove it all until we have an implemented use case somewhere.
Good point about the performance on panel animation,
I think Hamish has some JS-related optimizations there,
so we'll see how jarring they still are first.

On 1/11/2012, at 12:00 AM, Naomi Guyer <notifications@github.com> wrote:

> In admin/scss/_mixins.scss:
> 
> > @@ -101,5 +101,92 @@
> >  	*:first-child &{ zoom:1;}
> >  }
> >  
> > +//**----------------------------------------------------
> Good point, we probably don't need them. They are general purpose, but if we have no need for css animation, then we don't need them here. More complex animations we'd probably have to use javascript for anyway, as a lot of the actions are already javascript*.
> 
> I'd be tempted to leave the duration mixin in there, and remove the more complex animation keyframe stuff (and delay, as the use case isn't as general purpose). The duration mixin lets you do simple ui stuff like transition hover changes easily. It might be worth using this effect on our button hover? It might not be worth it though. I can remove them all for now if that's easier?
> 
> (* I think it would be nice to animate the expanding and shrinking of panels, particularly the preview as the change is a little jarring.)
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
I would say you don't need to declare this here, since it is set properly in the init function below.
How do we reproduce this behavior in different adapters? Are we relying on this to have `enabled? == true` when the experiment is first created?
Yes in our case it should be empty. 

There should be no OS or IDE specific entries.

But if this make the merge of kaik changes possible let it like it is.
Would be great to have this in the same line as the WHERE
Can you comment why this needs to be !important?
Ah, I see. Good catch!
On Mar 11, 2012 7:23 AM, "Matt Beckett" <
reply@reply.github.com>
wrote:

> > @@ -193,7 +195,7 @@ function plugins_add_submenus() {
> >
> >       if (elgg_is_logged_in() && elgg_get_page_owner_guid() ==
> elgg_get_logged_in_user_guid()) {
> >               elgg_register_menu_item('page', array(
> > -                     'href' =>
> "/$plugins_base/developer/$page_owner->username",
> > +                     'href' =>
> "$plugins_base/developer/$page_owner->username",
>
> That's true for a relative URL, but in this case $plugins_base is a fully
> qualified URL from elgg_get_site_url()
>
> so the link ended up looking like <a href="
> http://localhost/http://localhost/elgg/plugins/developer/admin">
>
> I don't think that's an Elgg bug is it?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Elgg/community_plugins/pull/4/files#r543408
>
Are you sure ``$name`` has the correct type as ``integer|string`` is not a sub-type of ``string``?
we should cap it with strndup
Wenn es ein Singleton sein soll, bitte threadsafe machen. Am einfachsten mit monitor
haben wir bei den andern doch auch so? Oder?
Edge case: what if the only versions uploaded are pre-release ones? Should we use the description from the most recent one then?
It's the docblock from the ```StorageInterface```
in that case, simply use

    if (!empty($array['articletext']))
should be split up in multiple `it`s.
What if there is no elements in `stations_nearby_all`?
I am not sure if we need to store this. Because query by ID is very fast.
Good. Then clean this.

Yes, I know. It is not easy problem. So I am leaving it a bit to "steam".
OK. This will have to be improved. It is mostly duplication of form.html. This is not good. It should be done in extendable way. But I will do that.
Empty line before this.
Why is this here and capitalized?
which means that you would have configured 2 different seed providers. Who would do that?
@stof the problem with shortening the class name is IDE. Some of them have problems with sortened or aliased classes.
I'd suggest to keep the FQCN
Cred ca poti face target.toggleClass('icon-minus') si sa scapi de if-uri si san u dai removeClass.
does it need to override?
Possibly use the object directly
It is worst to have it in C# because the Json serieliser send the messages in MapElement[] and not as TransferMap
Line too long for github
I think we discussed it and I probably have kept it this way. That was reviewed before, it just shows up here because there's commit from merging with cutting-edge. Not sure if it normally show up or if it's because I hand-fixed something after merging (but before committing the merge).
We have `contains()`.
Why not use the plain ngx_palloc()? It chould do the aligned work.
I didn't like to call this part highlights, but didn't come up with anything better than calling is "basic"
improved numerical precision and maybe efficiency:
define this as log so we only need the sum, and we don't have to take the log in the cv_ml loop.

But I don't really understand this part and the connection with cv_ml yet
Should be using Log instead of System.out.println
Will add null check.
Weird.  Isn't doxygen ignoring our other underscore-prefixed classes?  I thought it treated them as private and ignored them.  I could be wrong though.
Worth a FIXME to say why this test is enabled=false
Hm, we should delete this OldJmxSensorAdapter (in a separate pull request). It's only used by the sandbox DerbyDatabase currently...
if it times out we throw an exception to that effect; if that exception is swallowed by the test, i'd say it's a problem in the test, not something we should log info here.  have tidied it a little bit.

don't mind whether exceptions handled or thrown ... it seems clear and okay as it is, even if rethrowing might be better
I vote for fixing it. It won't affect any old apps (unless we silently ignored their "1" string because we didn't issue warnings, in which case it suddenly starts doing what they expected it too..), and only helps people in the future :)
Adding an extra line here about what `extra_params` is would be nice.
I see the credits are still here, possible to move to a comment in the source code? Also, the license for the old lady and dog photo is "Attribution-NoDerivs 2.0 Generic (CC BY-ND 2.0)"... does this count as a derivative work? (no idea). I think attribution in the CSS or HMTL is probably sufficient, though.
I used a document referencing a File document to work around that. This also allows you to have a new path/id when you upload a new file (think of http caching ...).
Seems like this should be a .ui file :)
minor: requiring list here and tuple for pairs seems fragile and un-pythonic to me... why not accept either in both places?
Nope - I'll fix it now.
Must go into controller for User or Friendship.
¿Por qué español?
replace these two lines with:
user.visible_message("\red <B>\The [src] was hit with \a [I] by \the [user]!</B>", "\red <B>You hit \the [src] with \the [I]!</B>", "You hear an object hitting glass.")
Mmmmm why would you get rid of this one?
Np, just would like to get rid of (null, null) arguments. Thoughts?
@caseysoftware Do we maybe want to change the text here based on this change?
Das sollt/wollt ich im Wiki machen ;)
Я похоже забыл ее убрать.
А, это происходит не одновременно.
Чтобы не делать таких заглушек - можно сортировать по умолчанию по чему-нибудь.
Yeah, that was definitely one concern of mine.  On the other hand, ssl_verify_peer is pretty much useless at this point.
A comment for this would be nice. Same for all POMs that define this.
`newForAllTabs`?
this logic should happen in the test not hidden in a page object
Look for amd, I use --with-external-libraries --with-internal-ffmpeg and should use the internal file. I I only select  --with-external-ffmpeg in XBMC it should use external include same as if I only specify --with-external-libraries
Titles and authors for these references?
> Sorry I forgot to clean up. I have pylint turned on, that should give the same result?

Yes
Not sure I like "graph" here. Maybe rather "affinity".
Damn (because I didn't think of that)!

You are absolutely right, though I wouldn't make it mandatory, but let it default to text/plain.
Additionally I might also want to add the Content-Transfer-Encoding Header?

(Maybe in a second step even multipart messages?)
Hrm, not really... Just wanted a good representation, because these objects are included in `X509Certificate`'s `toString()` output.
Yeah, good call-- that's the better way to do it.
We should also remove the datasource name from the persistence.xml and set the JPA default datasource name.

For the test, standalone.xml should contain:
<subsystem xmlns="urn:jboss:domain:jpa:1.0">
            <jpa default-datasource="java:jboss/datasources/ExampleDS"/>
</subsystem>
Not sure this is still the case, but using `@author` used to be generally discouraged in open source projects. If somebody really want to know the author, they can use `git blame`.
Agree with changing to "origFoo".  As for whether or not we need all these -- I wrote up that section thinking that we might, considering what we expected to be overwriting (see README.md).  Not all of the code is in, I think, so we may still end up using some of them.  I'll clean it up then.
BTW, il faudrait aligner ce tableau correctement
That's fine for now. I may merge them if I get some time.
Changed code to set 'rows' in call to performSort
it's a tab, and am a bit puzzled where they're appearing from! (or how they slip through the commit git checks)
a pytest_runtest_protocol hook call has the first non-None return value as result
so if one hook implementation bails out and returns None, the next one is called
suspect it is mistake in master, must be isset($fieldMapping['gedmo']['translatable']) instead of in_array
Here I'd expect `<tr>` not `</tr>`.

This is repeated several times.
CS: no need for line break
The code you wrote still looks based off of the Schaul plugin. I think we should still keep the license around. Read the relevant part of the license:

> Permission is hereby granted, free of charge, to any person obtaining a copy
> of this software and associated documentation files (the "Software"), to deal
> in the Software without restriction, including without limitation the rights
> to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
> copies of the Software, and to permit persons to whom the Software is
> furnished to do so, subject to the following conditions:

> The above copyright notice and this permission notice shall be included in
> all copies or substantial portions of the Software.
This should be SPI_CS_23K256, shouldn't it?
It's not necessary to slice it here as you'll slice it later as well.
Yeah, Python is the enemy... – but still better than Java :P
Doesn't this couple this cookbook with our cookbook-networking cookbook?
It would be good to use braces for `${name}` as well, in all files. I'll merge this already though.
looks like this is missing the test that says the fragment should now have include/exclude
Version isn't needed anymore. It applies for all these SAP related modules. I'm not going to annoy writing it more times.
I'm no expert, but `-fopenmp` doesn't seem to harm compilation and linkage of test code without openmp installed. No idea if there are subtler errors or even inefficiencies introduced by compiling this way. Presumably not if OpenMP is to be found (or not) later by dynamic linkage. But again I have no idea for sure.
We shouldn't be using @ManagedBean at all in a CDI example.
The class is named `Material`, not `Material.js`.  This is a breaking change so put it in the breaking change section and provide a before/after code example.  Use `[]()` markdown to link to the wiki.
When doing the real Columbus view design (I'm about 30 minutes into it at this point, really), we can consider an array of cameras indexed by mode to avoid `switch` statements like these.
Given the check in `update`, there is no need to check `show` here, right?
While your in here, do you want to make `_materialCache` a file-level scope variable instead?
Okay, I'll admit, that was a very good use of my time.  The tests are now much cleaner and clearer.  I ended up breaking up larger tests into smaller tests in order to better explain what was going on.  I think everything is much better on the whole now.  Hopefully that's everything, let me know if any other changes are needed.
"local private scope" is a bit of a confusing name, though I know what you mean.  It's neither local to the calling function, nor "private" in our usual underscored sense.  How about "module scope" since the idea is vaguely analogous to file scope in C++.
I think this confuses the Cesium Widget, who doesn't know that we turned off lighting so its lighting button has the wrong state.
This is going to get returned by `getPositions`.  This is inconsistent with `configureExtent`, who sets this member to `undefined`, since configure functions don't necessarily have positions as `getPositions` knows them.  You are using `_positions` as both one array of positions and an array of arrays of positions; even though JavaScript let's us do this, it is not a good practice.  Instead, introduce a new member, perhaps `_polygonHierarchy`, just like `configureExtent` introduced `_extent `.
Make?  Match?
"but found one with name" rather than "but found with name"
This should be:

    logger.log(__('%s was successfully set to %s', key.cyan, value.cyan) + '\n');
I am sympathetic to this. Generally, in Java, your constructor shouldn't actually *do* things. However, this constructor doesn't actually do things either. I.e. "run" is a misnomer, because it just sets up a bunch of event handlers. Perhaps you can suggest what parts of the constructor you'd like to see move into a run method. Perhaps you'd like to see everything in the constructor moved to a run method?
Wel this is not the behaviour I want,

I want toolcahin.MPI_TYPE to return either `"OPENMPI"` or `"MPICH"`

Can this magic be documented somewhere? I'm not sure I like magic...
s/StoppingPolicy is/$(D StoppingPolicy) is/
useless comment, better just call bck backup!
This line is too long, but you already knew that :)
You should check for null here before passing to `parseId`. The result can be null.
Pleasing to add Javadoc, and say what happens when `key` is not found.
this bean is never exposed anywhere.
The class is internal.
I think it's OK in 2.x.y
for the interface, handle at own will, just wanted to point it out

for the class. I rather though that the EO model is something only known to the enhancer. But the code would be placed more logical there. Think about something like: EOModel model = EOModel.isEngeneeringModel(model, service1, service2,...). model.doSomeMagic
:ttl?
Some tools probably won't have a nice icon for launch - so we should set this field to be optional.
the processor is not used
This is wrong. (Generally speaking I like to preview every file before I commit it, and if I've left something like this, I clean it up before committing.)
The indent level needs to be the same for all "columns", or our style check tools will complain.
Lukas: Do you want me to rewrite tests? Or you will commit yours.
'0 questions unanswered' reads a little bit funny. You might want to add:

`zero: 'No questions answered'`
Yup - agreed.
OK, I just checked the Oases sources, and you really do need the Velvet source dir.

Even worse, you need access to the Velvet build dir, after that you've just built Velvet, because Oases apparently needs the object (*.o) files from the Velvet build dir. This is really nasty.

This basically means that you'll need to build Velvet first, and then Oases, but in a single EasyBuild run. Ouch. It's the first time I've run into something like this, and I don't like it one bit. :)

The good news is that Velvet doesn't require any configuration, and just uses make. So, prior to building Oases, you should simply run `make` first in the Velvet source dir. 

So, the summarize: you need the Velvet sources to build Oases, Velvet is not a build dependency. You better include Velvet as a true dependency, so the version you used the sources of and the Velvet version you'll use together with Oases after you've loaded the Oases module is exactly the same.
In the `build_step` of Oases, just change to the unpacked Velvet source dir, and then set `VELVET_DIR` as explained in https://github.com/hpcugent/easybuild-easyblocks/pull/53/files#r2357138 .

I hope this is clear, this whole discussion was a bit confusing. ;-)
I do see your point, and I agree it's probably safer. It takes away a lot of convenience though when using FOSRestBundle that means you can't just configure the serializer and then return an array of data, you'll have to create a view and give it the serializer (which isn't even possible right now, but fixable). Anyway if that's the only problem you have with the PR I happily fix it.
I broke those maps out to SuiteMap and TagMap classes that know how to handle their own importing and mapping.  I just pass those classes where I need them.
"showing, that a method is doing much more, than it should do"? Do you mix it up with "number of lines in a method"?

However, my point was to set a limit, that is higher, than every possible (reasonable) convention. Maybe my values were a little bit to optimistic: 40 characters for identifiers, 150/180 characters max line length? The idea is, that every project specific convention can tighten this rules without violate PSR-1. Allowing lines longer than 180 (?) characters seems as illogical as mixed line-endings and even more illogical than breaking the "where to put the braces/paranthesis at"-rules.
druntime was created with the goal of being the standard runtime, not the DMD runtime. It was specifically created to reduce forking! It would be horrible if it had the reverse effect. Ideally, only the stuff in src.rt should be different between compilers. That's probably far from true at the moment, but I hope we can move towards merging all the other differences.
There is a constant temptation in DMD to accidentally make things x86 specific, and it's really hard to avoid right now. I have a dream of having GDC on ARM running on the autotester...
Just put the @trusted nothrow once at the top, and it will cover all of them.
Add a note mentioning the bugs if this is ever reached.
Guess who forgot to stage a file?
Auto-magic is the best....
Seems like its been deprecated for a while. Should it be removed already?
Let's leave it in for M2.

Personally, I feel it's pretty obvious that it would only apply if we are actually sending data, but it must have come from somewhere.
on a philosophical django tangent... we've had the idea of enforcing different levels of data validation. Like an unpublished article doesn't need a slug, but a published one does, so should the slugfield be required? As I typed that out... I think the answer is yes. before the save() hits, you should just pre-generate one, even if it's gibberish. Likewise here, we can assume the implementer has figured out how to get the latlong already, or decides to use a dummy latlong and then fills in the real one later (like how Google maps used to default to the geographic center of the continental US).
(And it is smart to simply have multiple directories for each branch, so that you do not have to deal with switching between branches in working directory.)
some of this stuff has gotten a little out of date by now:

- we have a number of optimizations for the hbr topological analysis steps that will improve performance by at least an order of magnitude. some of it has already been implemented, and the rest is pending review / validation.

- we aren't really looking for suggestions just yet on the feature adaptive part as this is our next big ticket implementation item that we need to generalize from the prototype code.
we hadn't removed this before?  hah!  I'm surprised it still worked!
on code en whitespace.
Oh, sry. The second commit was accidentally pushed.
I just wanted to push the editor.field.focus();

---

I had to override this, we are using Sencha Architect and it doesn't allow import of custom components. So the only way to get it working was to create a new DataView class (WorkOrder.view.GridView) and than make an override with your code for this class.
alternative syntax here too
I think it is an aspiration to keep the portal and CLI on par functionally. In this particular case I think we decided the option will be used so infrequently that it should not matter how heavy it is.
I've tested with the url given in the issue description and it produced the expected result (Twitter's search page with lots of results about 'screenshot')
>    Same here, but this is an exception because io is also a stdlib module.
>    The same import form is used in the scipy docs.

OK, I buy this
the fit method
What happens if trackKeys is undefined or empty string here?
Maybe you're saying you want a change to the HTML?  If so, can you add a comment specifying the change to the HTML you would like?
The Twilio Response is UTF-8  encoded.  Does it hurt to just add it to the getBytes method?
This is adding a FileSystem function into the File class. Is this intentional?
Is this really JSON, or is it an object. If it were JSON I would expect the user to have to call JSON.parse
With the change, this warning doesn't make sense any more. Any clue what the "commit problems" were?
again, mention of `this` is excluded here
This interface is going to be removed in 3.0.
All handling of creating and managing connectors should be done with the "ConnectorManager".
OK, thanks for the info. Unless you object, I am going to submit the code as is - and take care of the static method in one of the following checkins. The main idea, however, still remains the same - HTTP errors get translated into exceptions by a handler; when exception occurs, further tasks in the chain are not executed.
Take note that these are for jsdocs.  Add commented line above to denote this
If I get a general array of fields (names of simple fields + association mapping fields), then in the `ModelManager::getDataSourceIterator()` will need determine the type of field: simple or association mapping, in order to add LeftJoin for each association field. If this is not done we have Lazy Loading for each Entity, and when we export, we have a lot of queries to the database. 

[In this code](https://github.com/suncat2000/SonataDoctrineORMAdminBundle/commit/61d8ea7b3fd1afe7ad3d76e21178e9ebd0d62b6d) in the `ModelManager::getDataSourceIterator()` we check `$associationMappings` argument, if `count($associationMappings) > 0`, then modify the Query.

In addition, `ClassMetadataInfo` has two methods: `getFieldNames()` and `getAssociationMappings()`, so I thought was better to make an analogy.
can we use QSNotification... instead of Logs? They're sooooo old school ;-)

I think we had an unofficial chat about this on IRC and we both agreed it'd be nice to make things more obvious to the user, yeah?
These are fair points, but what about people that register in advance and don't show up to the event? With limited seating, these folks can crowd out others who might otherwise have shown up at the event and helped make it a success.

There are other ways to encourage people that have registered to attend in advance to show up on the day of the event, and charging a small fee is but one of them. I'm simply suggesting it be considered as one possible approach.

The job of a hackathon organizer is to ensure that the proper number and mix of people show up on the day of the event. Most people do have $5-10 dollars to spare for a weekend hacking event, and most people do have some form of credit/debit card. While not right in every situation, it is reasonable to charge a small fee if it keeps people from registering in advance but not attending.

I've used this approach myself for a few highly successful events. It works.
Though the sensor will now be the same instance for every Alarm instance.
Do you want the unicode check here (as you do for RelatedContentManager)?
`if isinstance(index, str) or isinstance(index, unicode)`

or better - `if instance(index, basestring)` - will catch both
use 7 now
又失误了.. 把这行去掉了.