labofsahil/github-event-dataset-2012 huggingface dataset:
It's there so that compiles on the mac don't take forever. I didn't want to disable this warning globally, as it could conceivably be useful.

It's not really related to the rest of the commit though - I can take it out if you are offended by it.
Ah, yes, that is the intention.

```numRows``` will probably not be bigger than 32bit, but the temporary result of the multiplication ```(nodeid * numRows)``` might. The end result, after division by ```nodes``` won't be bigger too (even less likely, in fact), since ```nodeid``` is always bigger than ```nodes```.

So, following standard C promotions, all temporaries will be 64 bit but the end result will be truncated to 32bit again. I hope compilers won't complain about loss of precision, since this is a pretty standard C promotion, but in case they do, we can cast it back to ```unsigned``` at the end.
So we're HTML5 now. Why not write ``<meta charset=utf-8>`` instead?
Why not 'whiskerboard'?
There seems to be a formatting issue with the license header.
The cleanup issue is a bit knotty, as it is never clear when the file has been picked up by the client.
One improvement would be to default the 'temp' directory to ```tempdir()```, which, at least o n *nix systems, typically gets reaper-ed, and wiped on restarts.
I  checked and my right trim multitine is enabled. I can't tell if it has solved that issue.
It'd be cool if there were an easy way to add a linter to our test suite. It'd also be cool if I were better at checking this -_-;
New protocol? ;)
Folder list should be always retrieved first, but is possible to go around it, so IMO we should also show empty list in this case to no confuse the user with messages that will disappear from the view after inbox is retrieved.
You seem to be using this option in a manner equivalent to the meaning of Actions.
I think this is more of a problem with the x64 mimikatz bin... I've fiddled with the 'bin' path and it gets past this exception (but causes the spawned process to crash) if I use cmd.exe etc. The current exe from mimikatz website has no icon on the x64 version so perhaps its corrupt. Will have to try later.

On Win x86 test it works (but doesn't spot my accounts with empty creds): 


[*] Reloading module...

[*] Running module against MSFDEV
[+] Saved passwords for admin norm user
[*] Post module execution completed
msf  post(mimikatz) > creds

Credentials
===========

host          port  user   pass                                                               type      active?
----          ----  ----   ----                                                               ----      -------
192.168.1.45  445   admin  password                                                           password  true
192.168.1.45  445   admin  e52cac67419a9a224a3b108f3fa6cb6d:8846f7eaee8fb117ad06bdd830b7586c  smb_hash  true
192.168.1.45  445   norm   aad3b435b51404eeaad3b435b51404ee:31d6cfe0d16ae931b73c59d7e0c089c0  smb_hash  true
192.168.1.45  445   user   aad3b435b51404eeaad3b435b51404ee:31d6cfe0d16ae931b73c59d7e0c089c0  smb_hash  true

[*] Found 4 credentials.
the `compact.join(' ')` does work, any opinions on why (or if) its better than `join(' ').strip`?
Okay, I like `blendColor` then.
Further, maybe this function should be renamed to `loadText` to match `loadImage` and the future `loadArrayBuffer` in the imagery_layers branch.
line breaks, and elsewhere in the file as well
I think the default here is actually transparent, so the parent color shows through.
optional + default when animated=true
you can use the `run` command here, it `2>&1` by default, it doesn't look like we need the output of this command while building
Remove. Or make actual comment from it?
lets  change this line to `Take appropriate steps to rename the @pyconindia twitter account and use @pythonindia for all notifications of user group meetups and everything else. `
OK, remove this file from git and add it to .gitignore.  See http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/ for reference
why changing that?
This comment isn't going to show up in the docs. It's the first `static if` which wins inside of a template. So, you should move the finite version above the infinite one.
plans to revamp placement of "{" soon?
ddoc uses `D_PARAM` for function parameters. It might make sense to use it on something that is an actual function parameter if it doesn't get highlighted on its own for some reason, but it shouldn't be use on stuff other than parameters. `true` simply needs syntax highlighting. It might be an argument, but it's never a parameter.
so I shall make it optional through a boolean value or something that 
can be set in the edb config file for now?

Am 2012-10-21 20:27, schrieb Andreas Pieber:
>>       public Long commit(EDBCommit commit) throws EDBException {
>>           if (commit.isCommitted()) {
>>               throw new EDBException("EDBCommit is already commitet.");
>>           }
>> +        if (commit.getParentRevisionNumber() != null
> well, the ida is that the entire checking is optional. maybe we could use the config admin to validate this
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openengsb/openengsb-framework/pull/784/files#r1899004
>
I think auch decisions need to be done by the objects themselves. The trick
is rather creating the correct objects :-)
On Oct 8, 2012 10:20 AM, "Felix Mayerhuber" <notifications@github.com>
wrote:

> In
> components/ekb/persistence-persist-edb/src/main/java/org/openengsb/core/ekb/persistence/persist/edb/internal/EngineeringObjectEnhancer.java:
>
> > +import org.openengsb.core.edb.api.EngineeringDatabaseService;
> > +import org.openengsb.core.ekb.api.EKBCommit;
> > +import org.openengsb.core.ekb.api.EKBException;
> > +import org.openengsb.core.ekb.api.ModelRegistry;
> > +import org.openengsb.core.ekb.api.TransformationEngine;
> > +import org.openengsb.core.ekb.common.EDBConverter;
> > +import org.openengsb.core.ekb.common.EDBConverterUtils;
> > +import org.openengsb.core.util.ModelUtils;
> > +import org.slf4j.Logger;
> > +import org.slf4j.LoggerFactory;
> > +
> > +/**
> > + * The EngineeringObjectEnhancer enhance an EKBCommit object with additional models which need to be updated or enhance
> > + * inserted models based on the Engineering Object concept of the OpenEngSB.
> > + */
> > +public class EngineeringObjectEnhancer {
>
> hmmm ... again something I need to think about how to make that properly.
> I mean the interesting point here is: what makes sense to be put in this
> EOModel class. And also it will be a little bit harder in the enhancer to
> do bookkeeping which models are EOModels so that they can be casted to that
> or if they were already enhanced like shown by you. Or maybe I just think
> the wrong way and you don't mean it at all like I imagine : D
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openengsb/openengsb-framework/pull/776/files#r1782592>.
>
>
remove labs delegation
It is how matt told me I should do it.
Good point. I'll change the title.
I think that we should remove `storage accounts`, `affinity groups` and `locations` as they were part of old release
nice fix.  Really should be in i18n.  do NOT correct - needs to be updated in CB2
Same as with get_file
Get rid of this one, only use the `-with-extensions` one, but get rid of the version suffix.
hay que poner el asiento nuevo de lanchita xD
Now supported annotations Groups, SerializedName, Version, XmlAttribute, XmlList, Inline, Since, Until, VirtualProperty, XmlMap, XmlValue. For other annotations it's don't need
bin nicht sicher... aber ich vermute, dass das Repo ein leeres Array zurück liefern würde.... was auch != null ist!
Schau dir die Funktion empty() von PHP an.
why mapped superclass? what would happen if we allow to store a core symfony route in phpcr?
good point.  I use a factory now.
Did you want the css guidelines applied to the entire file, or just my changes?
Should probably create a simple variable and assign it to `tempDocData[tempItem._ref][cell.id][cell.type]` so you don't have to repeat it everywhere here
Ah OK, it just looked like an idiom therefor the question.
setting __the__ CachingConnection factory's 'cacheConsumers' attribute...
If that's "exactly how it was" before, but you decided to take a different approach when moving the logic into a ConnectionFactory wrapper, I assume there was a reason. So I'm curious... did you discover something wrong with that approach then?... or was it just not possible in the wrapper for some other reason?
What to throw here? I think we can just re-throw what is caught here.
I think expire encompasis the idea of "be automatically removed from the queue". To be consistent with the comment below, it will be awesome if we can remove this part.
Diagnostic ?
Let's stick to camelCase. There's a few other spots that need to be changed too.
this should be RedHat
Let's us on-left and on-right to stay consistent with Twitter Bootstrap
+ implies referenceable
Это для тех, кто может быть вставляет прямо в лэйаут <script>
Can you add the valid indentation here ?
`msg.messages[i].messageId` here then
cred că poate sa dispară linia asta comentată
I think he's saying that in the previous line you have actualcount set to array.array.length / stride; so your actual test is
```actionscript3
if (array.array.length < (offset + (int (array.array.length / stride) * stride))) {
```
Here again, implement `IEnumerable<T>` and change this method to return `IEnumerator<T>`.
Nitpicky: It's ok to capitalize KML in comments, just not in identifier names.  Also, end summaries with a period.
move to util or presenter, and add test
Reuse the grid instead of making a new one each time.   Also, make sure that we set the state of the previousVersions table to closed.
It's the same file addition as before. I just had to use renameSync because copyFile from fs doesn't allow you to specify the output file name.
Should the option be called `binary` or `binarize`? I prefer the latter.
Are there really 8 different sources for the map center? We should really unify these if so.
save some bytes by rounding these to 1.8s?
I think I like this, but we should probably write down somewhere what our guarantees on message SUIDs are. Specifically, I think we guarantee that they're of the form "accountId / folderNum / messageId". I don't know if we need utility code to manipulate the SUIDs, since there are only two protocols, so we'll probably only use each utility function twice.

Perhaps when we move all the files around, we could add a README to data/dep/mailapi that explains some of the details like that.
can we use `None` instead of an empty string for the default value here (and on lines 2075 and 2081 below?)
>
> The conf dir can be located using win32-dir gem which defines the
> Dir::CommonAppData constant, which does the right thing on 2003 and 2008.
> Facter already has this dependency, so it should be straightforward to add
>

This is what's a bit weird.  The fact "root_home" is meant to model the
home directory of the root user account.  It's not really meant to be the
Puppet configuration directory.

I'm not sure if we're actually using root_home anywhere in the PE modules,
but if we are another option is to simply disable this fact for Windows and
be more defensive about it not existing on all platforms in the PE modules.

-- 
Jeff McCune
I dunno - this isn't literally 'root_home' ... root home is the Administrator's home directory IMO. Weither this is useful or not is another question :-). But facts for these common items for Windows would be sweet :-).
Either way works, but we don't display any information from a single `PatientProcedure` record. Instead each row is a `Procedure` with summary information from `PatientProcedure`. Therefore if we use `Procedure` as our base, we can then in the view do:

```ruby
- @report.procedures do |procedure|
  = procedure.full_description
  = procedure.total_count
  = procedure.total_value
```

Which seems way more natural then:

```ruby
- @report.procedures do |patient_procedure|
  = patient_procedure.procedure.full_description
  = patient_procedure.total_count
  = patient_procedure.total_value
```

Does that make sense?
Hm, I thought it's two lines between top-level items.
I meant to ask, it can't have 5 choices?  Or does that break the computer that scores the results and notifies your preferred college and any future employer?
RELEASE_TO_NIL does the exact same thing as [scrollViews release], except it also checks if the variable is nil before releasing it. I have already checked if the variable is nil.

    #define RELEASE_TO_NIL(x) { if (x!=nil) { [x release]; x = nil; } }

https://github.com/appcelerator/titanium_mobile/blob/master/iphone/Classes/TiBase.h#L72
can you drop in a few lines in the javadoc for what this class is and why it is needed?
Missing `f` suffix to make this a float constant.
Please, do not use spaces - replace it with tabs. Just like it is on line 32.
Opened blackberry/BB10-Webworks-Packager#224 to track
Referring to JtaTransactionProvider.class does that trigger loading of JTA classes on the client side? Or can it be ran without javax.transaction on client classpath?
I missed this when i ported over this portion of the code. I will do some more cleanup.
I have nothing against changing the code, but just so you know other tests just make a mockedQnx and expose it to GLOBAL.qnx and GLOBAL.window.qnx
Instead of calling qnx.callExtensionMethod directly here, let's use window.qnx.webplatform.getApplication().invocation.getRequest()?
is this ever deleted anywhere?
What's the use case for this function?

Is `set_item` really the best name for this?  `set` generally does not imply remove if.  Maybe `toggle_item`?
If this is going to stay as a Class (against my opinion), then this should (in another of my opinions..) be in it's own file in a folder of the same name (ex `webviewFactory/WebView.js`. This will make this file a LOT shorter, easier to read, and also place WebView in it's own (modular) file.