labofsahil/github-event-dataset-2012 huggingface dataset:
```ending_encounters = Encounter.where('end_time IS NULL OR end_time >= :end_time', {:end_time => 12.hours.ago})
patients = ending_encounters.map {|e| e.patient}.uniq{|p| p.id}
encounters = patients.map {|p| p.encounters}.inject{|a, b| a + b}
```

Or something like that

At least not the SQL
Okay, I think it will be solved once custom sample test page will be created.
While this works, would it be possible to follow the flow of the if-else statement to:

```groovy
if (this.androidSdkToolsRevision < 14) {
  return new AndroidSetup_r13(project)
} else if (this.androidSdkToolsRevision < 17) {
  return new AndroidSetup_r14(project)
} else {
  return new AndroidSetup_r17(project)
}
```

Should make this much easier to understand.
This setting should probably be changed anyway. See #533.
"But maybe we won't need this feature at all?"
does this file already exist, or did you forget to add it?
Ты все правильно сделал.
Разве что, я бы .begin и .end на одну строку поместил — логически-то
это одно и то же.
Brace on separate line.
Need to set "timestamp_on_filename.tm_isdst = 0" as std::mktime() could fail as some compilers do not default the std::tm struct values.
Did you decide to stay with contains() rather than a regex?

Seems like endsWith() might be an option too, given that you are either matching the entire string (exchangeWithoutCorrelationKey) or the last part.
the reasoning behind having the Icons defined here is if there is not a handler to deal with the entity (currently only signs are handled) then this will act as  on / off. The icon for the signs is set in the main config file under "overviewerConfig.CONST.image.signMarker" which is referenced above. Do you mean that the icon / show entity objects above should be pre-set in the overviewer main config and then overridden (like regions used to work) by the user defined settings.py ?
What is the reasoning for this removal of a README file? Wouldn't it make more sense to just filter it out of anything that is accidentally reading it as a config file?
Use `null` for programmer specified undefined values. At least I have never get to the point to use `undefined` directly.
placeholder.children('canvas'), yes

On 29. 09. 12, at 12:32, Mitar <notifications@github.com> wrote:

> In tests/test_project/test_app/static/test_app/js/datastream.js:
> 
> > +        var add_metric = undefined;
> > +
> > +        if (options && options.metrics) {
> > +            options = $.extend(options, { 'datastream': {'metrics': options.metrics }});
> > +            delete options.metrics;
> > +        }
> > +
> > +        if (options && options.add_metric) {
> > +            add_metric = options.add_metric;
> > +            delete options.add_metric;
> > +        }
> > +
> > +        this.options = $.extend({}, $.datastream.defaults, options);
> > +        this.placeholder = placeholder;
> > +
> > +        if (placeholder.children('canvas').length > 0 && add_metric) {
> placeholder.children('canvas').length > 0 -> placeholder.children('canvas').length -> placeholder.children('canvas'), no?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
:+1: for the method though
Can you prototype this solution?

I'm aware of several pull requests you have issued, please merge in the code from master however first. I'm happy to mere it in, up until now it creates however merge conflicts for which I need substantially more time to resolve and then validate. If you handle this, I can merge faster.

--- Ursprüngl. Mitteilung ---
Von:Tobias Simon
Gesendet: 18-02-2012, 09:27
An: Meier  Lorenz
Betreff:Re: [qgroundcontrol] Various changes (#34)

> -                //                while (lastCompIndex->value(message.compid, 0)+1 )
> -                //            }
> -                //if ()
> +                                     // NOTE: Using uint8_t here auto-wraps the number around to 0.
> +                    expectedIndex = lastIndex[message.sysid][message.compid] + 1;
> +                             }
> +
> +                             // Make some noise if a message was skipped
> +                //qDebug() << "SYSID" << message.sysid << "COMPID" << message.compid << "MSGID" << message.msgid << "EXPECTED INDEX:" << expectedIndex << "SEQ" << message.seq;
> +                             if (message.seq != expectedIndex)
> +                             {
> +                                     // Determine how many messages were skipped accounting for 0-wraparound
> +                                     int16_t lostMessages = message.seq - expectedIndex;
> +                                     if (lostMessages < 0)
> +                                     {
> +                                             lostMessages += 256;

Hi!

This piece of code causes huge problems with out-of-order packets, which may occur when using UDP via some intermediate network nodes. We are facing serious loss rate peaks (up to 30%) each time a message is out-of-order, like in the sequence [2,1]. In this example we expect after 3 to be the next seq number, but we get 1, so lostMessages = -2. The if yields lostMessages = 254.. which is quite a lot!

Accoding to http://www.wireshark.org/lists/wireshark-users/201112/msg00030.html, wireshark calculates packet loss via (SEQ_N - SEQ_N-1 - 1), which is exactly the same as lostMessages = message.seq - expectedIndex in your code. The difference is that they allow this value to be negative.

So maybe one solution would be to remove the "if" or just set lostMessages to 0 instead of 256.

Regards,
Tobias

---
Reply to this email directly or view it on GitHub:
https://github.com/mavlink/qgroundcontrol/pull/34/files#r464652
there is no need to assign this to a variable. It can just be
```JavaScript
function apiInjector() {
```
Also, the convention is typical camel case. Classes start with an upper case letter, but everything else (including functions), starts with a lower case letter.
I think this filters on node-type, e.g. nt:unstructured, but am confused by the examples in the existing documentation where the value "a*" is provided, what does this accomplish? or is it simply "all node types begining with *a*"?
Is this supposed to be 'Understood, restraints released.'?
I just did, through I don't understand why... as all over methods are declared abstract exactly the same way. To keep sources consistent, all ``@abstract`` annotation should be kept or removed, without any particular cases.
I'm pretty sure I did this already in a previous commit ... is your branch up to date with upstream master?
Keep general methods used in the differents map providers.
well you're right.. ill change it
PrawnWorker return error (need to fix that worker)
'n'oubliez pas' -> 'Vous pouvez' ?
should we fail fast and error out here if the plugin was not found?
in sootiyaro dige jane ma nadin dige !!!!
On Android, there is no default value, or default value is undefined.
Scale event is not available in android
+1 to removing it.
This won't work on windows machines.
It is introduced by Jinja2 template scripting style ...
Indented too much
Jasné, na toto som zabudol. Bolo to len kvôli testovaniu. Odstránim to.
Did you forget to turn this back on?
Dont know if touch events will work here as the document object you want is in the different webview
Fixed that unit test that wasn't working before using Nukul's sample code.

Marco Di Cesare
Team Lead - BlackBerry Push Data Service
Research In Motion Limited


-----Original Message-----
From: Nukul Bhasin [mailto:reply@reply.github.com] 
Sent: Thursday, June 21, 2012 7:26 AM
To: Marco Di Cesare
Subject: Re: [BB10-WebWorks-Framework] Next device (#70)

> +
> +        afterEach(unloadClient);
> +
> +        it("execSync should have been called once for each blackberry.device field", function () {
> +            expect(mockedWebworks.execSync.callCount).toEqual(fields.length);
> +        });
> +
> +        it("hardwareid", function () {
> +            expect(mockedWebworks.execSync.argsForCall).toContain(execSyncArgs[fields.indexOf("hardwareId")]);
> +        });				
> +		
> +		it("version", function () {            
> +			expect(mockedWebworks.execSync.argsForCall).toContain(execSyncArgs[fields.indexOf("softwareVersion")]);            
> +        });
> +		
> +		//TODO is there a way to do this? Seems like toHaveBeenCalledWith only work is the function is called once

This test should read-
  it("readonly fields set", function () {                
            expect(mockedWebworks.defineReadOnlyField).toHaveBeenCalledWith(client, "hardwareId", null);
            expect(mockedWebworks.defineReadOnlyField).toHaveBeenCalledWith(client, "softwareVersion", null);
        }); 

This fixes-
1. somevalue is undefined, should be null
2. hardwareid should be hardwareId
3. version should be softwareVersion

---
Reply to this email directly or view it on GitHub:
https://github.com/blackberry-webworks/BB10-WebWorks-Framework/pull/70/files#r1025375

---------------------------------------------------------------------
This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.