labofsahil/github-event-dataset-2012 huggingface dataset:
It is. Is there an issue with that name? 

-----Original Message-----
From: Philip Schwartz [mailto:reply@reply.github.com] 
Sent: Tuesday, June 12, 2012 1:51 PM
To: Panagiotatos, Greg (RIS-BCT)
Subject: Re: [HPCC-Platform] fixes gh-2459 Clean up of docMacros.cmake to correct -j build failures. (#2526)

> @@ -15,4 +15,4 @@
>  #    along with this program.  If not, see <http://www.gnu.org/licenses/>.
>  ################################################################################
>  
> -DOCBOOK_TO_PDF( ${FO_XSL} SLR-includer.xml "ECLStandardLibraryReference")
> +DOCBOOK_TO_PDF( ${FO_XSL} SLR-includer.xml "ECLStandardLibraryReference" "SLR-Mods")

I think mods is short for modules.

---
Reply to this email directly or view it on GitHub:
https://github.com/hpcc-systems/HPCC-Platform/pull/2526/files#r969126
if (initRowSize > initKeySize) would be clearer
Good points. Will update.
you need to annotate it with ``@test`` otherwise PHPUnit will not run it as the name does not start with ``test``
not_if and only_if will run a shell command if you pass a string, so do that rather than calling system (see the above instance of this as well)
Is there any point in having that 'security.authentication_throttling' string in there if we're just going to hash it anyway?
GetCdnUrl ?
@christianv we should probably talk about whether we do little transforms like 'uppercase' in angular or in CSS - I think you and I are inconsistent there. Not a blocker.
what about beta and RC releases ?
Bug here if the filename contain spaces. should surround each filename by with quotes
`return unless value`
Our goal here is to compute any symbol that can be obtained from the input to `expandSymbol` through a chain of at most 3 `expand`. `expand` computes "expanded" symbols from its input, using relations from mixed in `SymbolsExpanders`:

  - This includes at least the reflexive relation : for any `s`, `expand(s)` contains `s`, so we need to filter duplicates.
  - `expand` is deterministic, so we never need to expand twice for the same symbol.

 At each recursive call to `expandSymbols(res, seen, ss)`, `ss` is the outermost layer of symbols, on which 'expand' has never been computed before. `nuS` updates that outermost layer taking only nodes with a `Symbol` into account. Details & explanatory schema in [Mirko's thesis](http://bit.ly/VLMt9T) p. 15.
Outside the `if`-block you can define
```python
upscale = (2**o - 1) / (2**n - 1)
downscale = 2**(o - m)
```
and use it here and in the `if copy` case.
I think alignment wins over PEP8-consistency for matrices:
```python
R = [[math.cos(angle), -math.sin(angle), 0],
     [math.sin(angle),  math.cos(angle), 0],
     [              0,                0, 1]]
```
What is the advantage in removing the double quotes here?
This doesn't really read well in English.
Not in generated doc.
This hidden argument should not be in the public interface.  Delegate to a private function, `_addImage`.
you might want to look at TextureAtlas.resize instead of creating a completely new texture atlas for when items are removed.
Object literal would be more consistent with the rest of Cesium, probably.  A rule of thumb I could make up right now would be: If you have more than one optional parameter, all optional parameters move to an optional object literal parameter.  

This is assuming that performance is not critical, passing parameters via objects is going to be slower.

This style is pretty common in Ruby & Rails too.
A model object should not be responsible for rendering shapes on the screen, right?
Hi Daniel, maybe we should delegate it to ag-sec like this https://github.com/aerogear/aerogear-security/blob/master/src/main/java/org/jboss/aerogear/security/exception/AeroGearSecurityException.java

Wdyt?
removing this will break https://jira.appcelerator.org/browse/TIMOB-9162
Should also include a cross-reference--this doesn't explain anything about what it does.

```
For more information, see the discussion of horizontal layout mode in the description of the [layout](Titanium.UI.View.layout) property.
```
In my opinion all language tags can be tried before giving up. Encoding issues should be fixed with this patch, but
e.g.: If one tag has an invalid language key (lets say wikipedia=dd:something instead of wikipedia=de:something) why shouldn't we try other available tags as wikipedia:en etc?

I was just unsure, why one should give up early; ... If you don't like it or are afraid of the traffic overhead, just drop this line of the changeset.
Are you asking me to write even more documentation?! :) I'll flip out. lol
I added this because I was getting errors when attempting to close a websocket that hadn't finished connecting. The same applies for the other blocks - I found it cleaner just to simply wrap it up in a try-catch and attempt to close it.
Symfony2 without a space
О нет, только не вайтспейс!
Good question. Personal style I guess.

Strange, `any?` seems to be slower anyway but that was not my reason ;)
If this is used only inside emplace, it would be simpler to simply define it inside emplace.
Need space.
Looks like you're referring to the AjaxLoader project? Those files are all in the Maven repos, so I don't think they're needed in the src. Also, I believe the src was a little behind the current Loader API 1.1 version.

Other files are not needed to build and run the project. I've been able to build and run it from src without those files or directories. They are either (1) war/webapp files that are not needed in a lib or (2) Files that are dynamically generated by the GWT compiler at build time, so including them makes the src bigger and means that we'll be checking in/out lots of changes that are just compile time baggage.
this link format is rather verbose--github usually adds the link automatically?
according to Bob, this might work, or at least they once had this working.  Ideally we should be able to do:

#import('package:webcomponents/list_component.dart');
nit: move field up top and give it a more precise type

might also be worth renaming it? perhaps "_removeWatcher" or "_stopWatching"
use ipa header from https://github.com/ipa320/setup/tree/master/templates
damn autogeneration...
so how would you get the map into the job?  i'm trying to work around the fact that this is a generalized lib and the log-weasel stuff is very specific.  but i'm about to stop caring before ken yells at me again.  he's very temperamental.
Ahh, bollocks, thanks for pointing that out. A new commit has been added to account for this.
i intentionally made it possible to configure the formats, because in many cases at least for html people will most likely prefer the current behavior.
Juraj, I guess these tests are accidentally commented out?
These are unique to this view.
If this class is required before `ci/reporter/minitest` is required it's going to explode because `CI::Reporter::Minitest` does not exist yet. You should require `ci/reporter/minitest` inside `minitest-chef-handler/ci_runner` and remove it from the line below.
and how do you detect when the event is fired?
this is awefully cut original version.
On 8 Dec 2012, at 18:52, Kenneth Hoste wrote:

> In easybuild/easyconfigs/o/Oases/Oases-0.2.08-goalf-1.1.0-no-OFED.eb:
> 
> > +
> > +name = 'Oases'
> > +version = '0.2.08'
> > +
> > +homepage = 'http://www.ebi.ac.uk/~zerbino/oases/'
> > +description = """De novo transcriptome assembler for very short reads"""
> > +
> > +toolchain = {'name':'goalf','version':'1.1.0-no-OFED'}
> > +toolchainopts = {'optarch':True, 'pic':True}
> > +
> > +velvet = 'Velvet'
> > +velvetver = '1.2.07'
> > +
> > +sources = [
> > +           '%s_%s.tgz' % (name.lower(),version),
> > +           '%s_%s.tgz' % (velvet.lower(), velvetver)
> 
> See https://github.com/hpcugent/easybuild-easyblocks/pull/53/files#r2357138 for a description on how to do it without specifying Velvet as build dep.
> 
> However, as you've pasted from the Oases documentation: you just need to have Velvet installed, you don't need access to it's sources.
> 
> Oases probably just needs header files that are provided by Velvet. In that caqse, Velvet is actually just a build dependency, and you should:
> 
> 	• not specify the Velvet tarball in sources in the Oases easyconfig
> 	• do specify Velvet as a build dependency (as you're doing now)
> 	• simple use VELVET_DIR = %s' % get_software_root('Velvet') in the Oases easyblock, as opposed to what's suggested in the comment I referred to above
> Is that clear? I'll also mention it in the easyblock pull request.
> 

I will try these things beginning of next week and hope that your suggestion from above will be enough for this particular situation. I think it would definitely be a nicer solution then what is there now.

Thank you.

> —
> Reply to this email directly or view it on GitHub.
> 
> 

Cedric Laczny,
PhD Student
 
UNIVERSITÉ DU LUXEMBOURG
 
LUXEMBOURG CENTRE FOR SYSTEMS BIOMEDICINE
Campus Belval | House of Biomedicine
7, avenue des Hauts-Fourneaux
L-4362 Esch-sur-Alzette
T +352 46 66 44 6398
F +352 46 66 44 6949
cedric.laczny@uni.lu     http://lcsb.uni.lu
 

-----
This message is confidential and may contain privileged information. It is intended for the named recipient only. If you receive it in error please notify me and permanently delete the original message and any copies.
-----
Couldn't this be done more concisely with `delta *= -1`?
This is a slightly odd way of writing things - I think it'd be better to write: "if ($ip) {" ...
```
The name of the property seems weird to me.
Since we use the subject for a couple of different properties (subject.text, subject.icon) we can probably reference it once.
Yes thats what I intended.
s/commutative/associative/
if-else convention, else on the next line
Yeah that's true. just added a check for that.
Thanks mate, changes  done and pushed..
Look at AbstractOutboundChannelAdapterParser If the empty check is only performed if polling consumer otherwise null will be sent to builder.addPropertyValue("inputChannelName", channelName); on line 67
good point, reverted.
style tip, use Java's foreach next time:
for (Map.Entry item : set) {
}
Not a big deal in this case though.
did you mean "sequentially" here? Right now it says parallel twice in my mind :)
We don't want to call process.exit in a plugin because it will kill the entire CLI! :-) Better to just return
`|` can be used for alternations and `\s` includes `\t`, so your code could look like `$_[0] =~ s/<\/?\w*>|^\s*|\s*$//g`.
Is "You need to have" a bit informal and chatty here?  "PostgreSQL must be installed and configured" is a bit more formal, and (in my opinion) more readable.
In fact: https://github.com/telefonicaid/notification_server/pull/119/files#L0R206 returns an array, not an object ;)
According to Authorize.Net SIM docs they are NOT ignored. I will update comments accordingly.
why do you test isMeeGo again? Isn't already declared at line 13?
This would just be FileInputFormat.setInputPaths(job). This is pretty much all input formats store the input path.
actually, just revert it back to what is at upstream/master
What exactly does the above do? Is it monkey-patching Chef's run_command to just return true?
Won't the code at line 57 return (0) if count(/project/file/parse_markers/*) is 0?
why? it is just few lines of code and pretty easy to maintain. 

As a rule a plugin has to implement `PluginInterface` so why dont we needed to test this rule?

What if I dont implement the interface? it will fail only on integration steps (to late as for me).  The other point for having this is "tests as documentation". For example you see that test: `couldBeConstructedWithoutAnyArguments` faild. It pretty easy to understand what was changed.

Someone can remove the interface while refacturing and you lost it whil review? where will it be caught?

BTW: I found my self writing such test when I do TDD.
But isn't a SchedulerMessage a message that the Scheduler handles? The controller still has to construct a SchedulerMessage to be able to send it to the Scheduler.
ops sorry
Je mettrais la phrase au pluriel "pendant que certains d'entre nous consacrent une partie de leur temps ou tout leur temps, les autres effectuent des prestations rentables immédiatement afin d'assurer les salaires de tout le monde" (moins familier que "facturent" ^^)
It is entirely possible -- and it blows up.  I've adjusted it.  Should probably adjust the other values in that function as well.
:(. I discourage 'import as' for anything apart from a few very standard cases (np, pl). It makes the code harder to read.
( Thank you Rémi, really happy to see someone improving the theme :-) )

Maybe we could use this syntax: “Articles (page 2) · Les intégristes”, what do you think?
So for ActiveSync, as discussed, we have the "show headers, then sync" strategy if the most-recent-sync is recent enough, and the flip-side is "sync, then show headers" when it's not or we've never synchronized.

It seems like we always want "show headers" to translate to:
```
      this.getMessagesInImapDateRange(
        0, FUTURE(), INITIAL_FILL_SIZE, INITIAL_FILL_SIZE, ...)
```

So then the problem is that having a non-null _curSyncSlice implies that we should possibly send headers directly over the wire, but all ActiveSync wants out of _curSyncSlice is for it to say "hey I'm synchronizing" and in the near-enhanced-future provide details for the progress bar.  In theory, we might also want to stream the headers for ActiveSync as we receive them for initial sync, but since it sounds like hotmail provides them in exactly the wrong order (oldest to newest), we don't actually want that...

More thoughts in another minute after I skim the patch more.
Why is this tabbed in? Same with below sample code.
Me resulta poco feliz el nombre de la función =S. Qué opinan?
I think it's enough to store it on the server then?

On Tue, Jul 24, 2012 at 4:34 PM, Peter Grman
<reply@reply.github.com>
wrote:
>> + * ownership. The AASTI licenses this file to you under the Apache License,
>> + * Version 2.0 (the "License"); you may not use this file except in compliance
>> + * with the License. You may obtain a copy of the License at
>> + *
>> + *     http://www.apache.org/licenses/LICENSE-2.0
>> + *
>> + * Unless required by applicable law or agreed to in writing, software
>> + * distributed under the License is distributed on an "AS IS" BASIS,
>> + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
>> + * See the License for the specific language governing permissions and
>> + * limitations under the License.
>> + */
>> +
>> +package org.openengsb.domain.requirement;
>> +
>> +public enum Status {
>
> I hope so, but the server would also need a property, so it knows where the process / workflow is currently.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openengsb/openengsb-domain-requirement/pull/1/files#r1225483
Not sure if I'm mistaken, but I think in Rails 3 / asset pipeline, `application.js` or the manifest file should only contain the require statements for Sprockets and is no longer supposed to act as the "dumping ground" for miscellaneous JS function calls. 

This is not to say that it should stay in the views, but I think the "best practice" way is to either create a "replacement" for the `application.js` file that contains application-wide stuff - something like `site.js` -  or else to create topic files for specific pages or parts of the app.

@jordanbyron, what do you think? Shall I create a `site.js` file and move the logic into that file? Maybe along with the other stuff that is in `application.js` right now?
Love the comments! :)
Also, @rcruz and I were talking and we were wondering if we should move some of this logic into the Packager. In the Framework you could simply check that config.autoDefer is set from framework.js. The Framework really shouldn't have to iterate through access lists and with this approach we would keep the `create` function clean.

Would be similar to what we did for enable-flash.

Thoughts?

`if (config.autoDefer) {
       _webviewObj.autoDeferNetworkingAndJavaScript = autoDefer;
 }`
souce? should be source? :P
substantially