labofsahil/github-event-dataset-2012 huggingface dataset:
Why are we attaching this to the object? 
I think this function should return a cleaned value: ie cast the variable to float using http://php.net/manual/en/function.floatval.php
Changed, committed and pushed.
Right, I think W&N would say that's the "least acceptable" 2-column solution (if at all).

They'd prefer "Text 2" and "Text 3" be swapped but since we can't do that we started floating things. How would `float: top;` on all `.span-all`'s by default affect numbering and layout?
In functional programming you would use "len: Option[Int] = None" instead of "len: Int = -1".
But I'm not sure what's better here. Maybe the standard Java approach with "-1" for "not present" is better here.

With Option it would look like:

     def read(cbuf: Array[Char], off: Int = 0, len: Option[Int] = None): Int = {
        val length = len.map{len=>math.min(len,cbuf.length)}.getOrElse(cbuf.length)

For someone not used to functional programming, this is far more complicated. Furthermore you would have to call it with read(cbuf,4,Some(3)) instead of just read(cbuf,4,3).

So just screw this comment and let it remain easy ;)
I did mention that in Michaels commit on the Serializer. I'd put that method into the FtanString companion object, I think it would fit there well.
Is it possible to add a ``@param`` annotation for parameter ``$username`` which defines a more specific range of types; something like ``string|array``, or ``null|MyObject``? Please also check line 58.
- Are you sure the variable ``$fields`` exists, or did you maybe mean ``$field``?
- Are you sure the variable ``$this`` exists, or did you maybe forget to declare it?
Are you sure ``$value`` has the correct type as "object|string|null|array|boolean|integer|double" is not a sub-type of "string"?
Ein Slaaaaash zu viel! :astonished:
This isn't OK.  If you look a little closer you will see that for the SysLog logger, there are different values for the priorities.  This is because they are used to derive constants which are expected by the SysLog function (http://php.net/manual/en/function.syslog.php).
Looks like you may have gotten a junk character in here.
Yay tests!

<img src="http://hivtest.cdc.gov/stronger/img/logo_stronger.png">
Well this is a semantic question, but as i see it, if NEEDS_TRUST is False, we don't care if we trust the user or not,
we could trust him or not, it doesn't matter. I don't think the method can_creator_be_trust should also answer to the question,
"is the module enabled ?"
No gettext?
Remover este construtor
let's collapse the 2 imports into 1 line
Agreed. I can create a better example and will push it -- thanks for reviewing this.
*Account* -> *account*
*reset your password* -> *reset your password in the case you forget it."
Oh I tried everything and went through django documentation and haven't found anything but django-admin.py. I didn't even try the simplest solution.
Sorry, this came out of a different thread; Homebrew can't determine the name of subformula correctly, and so it just uses the URL basename in the cache; in this case, the cached filename is "3819677a7f". So it's not necessary, but it does make the cache a little nicer.
even though the time string is presumably passed to some javascript, not to more python code?
Is this just for allowing a default order_id to be set? If so, I'd prefer to see it set in the options at gateway creation rather than via an attr_accessor.
is this the best you could do?
This flag has existed since Froyo, so we're safe here.
On Nov 10, 2012 2:09 PM, "Moxie Marlinspike" <notifications@github.com>
wrote:

> In src/org/thoughtcrime/redphone/call/LockManager.java:
>
> > @@ -42,6 +55,9 @@ public LockManager(Context context) {
> >      PowerManager pm = (PowerManager) context.getSystemService(Context.POWER_SERVICE);
> >      fullLock = pm.newWakeLock(PowerManager.SCREEN_BRIGHT_WAKE_LOCK | PowerManager.ACQUIRE_CAUSES_WAKEUP, "RedPhone Full");
> >      partialLock = pm.newWakeLock(PowerManager.PARTIAL_WAKE_LOCK, "RedPhone Partial");
> > +    proximityLock = pm.newWakeLock(PROXIMITY_SCREEN_OFF_WAKE_LOCK, "RedPhone Incall");
>
> Given what we're up to here, it seems like we might need to guard this
> construction (as you did with the reflection).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/WhisperSystems/RedPhone/pull/60/files#r2091555>.
>
>
It's actually more complicated than that: see https://docs.djangoproject.com/en/dev/faq/install/#what-python-version-can-i-use-with-django.
why explicitly return nil?  Almost never required.
logging the pid of the expired process could be more useful that the name of the watcher here.
Thank you so much! Sorry I didn't merge this sooner, but I've finally got a few hours to write express 2 and express 3 tests and land your patch.
Do you mean `((do_raw ? "raw" : "not_raw") + "_bound")`?

I don't get it though, why create these functions in this way instead of up-front?
Fixed throughout the file.
These two lines are legacy code to me ;)
That's the timeout handler, right? Isn't 10 ms a little on the short side?
I'm not sure how happy various tools will be with a version string like 'dev'. Maybe pick some small suitable release number.
a waste of memory, WLS and OLS as special cases?
if they are attached here as a temporary variable to make available in the iterations, then I would not leave them around as attributes for the user.

for example, I used something like self._temp as attribute to hold the values during iteration.
if they are for end user to know what was used in the program, it would need an indication that it was used for ci_kurt, and not for any other ones.

using dicts with dict updating sound also easier here, and lazy/cached attributes for the defaults ?
The result of `$this->selectable->matching($criteria)` should be stored in a variable as it can take a lot of processing and is user twice.
This seems like a pretty heavy-weight solution for this issue, with lots of backwards-compatibility issues. I know it would break quite a few test suites of mine. So I'd like to see another solution, if possible.
... and that fact is noted in the commit description. Please add some javadoc here for clarification.
WTF will "click here" do in a font tag?
BTW, you probably forgot to add your steezy meta generator tag to show off your awesome IDE
@pankdm прав, не стоит менять текст введенный пользователем. А вдруг он что то другое имел ввиду...
fixed typo, pushed
Can I get a second opinion on this?  I have to be honest, I don't think it's a great improvement over simply stating in the text that one of such-and-such size params must be set.
+1 for cleaning up imports
In this case it's sufficient if we just delete log file in ```basedir```.

Log files could also end up in the ```${project.artifactId}``` or ```target``` directories but they get deleted in the clean phase anyway.
Should there be a try-catch to always close stdoutG and stderrG?
Looks like there wasn't previously either, so this shouldn't block the pull request being merged...
I'm a little bit concerned about this statement: Notice that also strings have a length property, so `'abc'.length === 3`.

I suppose you want to check whether name is an array?

Then rather do

```js
if (Array.isArray(name)) {
...
```
Ah, I missed that. It is leftover from the example I was looking at and now I don't need it.
This looks like a typo.  Shouldn't it be `[0, 2, 3, 4, 4].should end_with [4, 4]`?
You are correct, `deepMerge` will not work with a collection. I can test that the current logic doesn't break when an array is passed in for a sideload response. I haven't built the logic to automatically attach a `ResourceCollection`, though plan to. Does this make sense? We can discuss more that the BBQ tomorow.

So I don't call `updateWithApiData` from within `updateSideloadApiData for two reasons. Currently, `updateWithApiData` actually contains the call to `updateSideloadApiData` which would cause in infinite loop. `updateWithApiData` also contains `beginPropertyChanges` and `endPropertyChanges` which should only be called once for the whole update process (the resource and all its sideloads). `updateSideloadApiData` is a recursive method, so this way it is completely within the `PropertyChanges` calls.

Both update methods do contain pretty much the same `deepMerge` line of code: `Ember.Resource.deepMerge(data, this.constructor.parse(json));`. I could refactor that into a separate method.
Correct me if I'm wrong, but won't the multimap returned from the method then contain only the values of the Address.getAddr() rather than the actual Address objects which is what we want?
Remove the two lines above.. they don't appear outside of a block.  (And are useless)
Please add success and error messages to the calls to executeQueryAsync.
Why was this part changed? Now you are going to update items which are allready in the offline cache. Or am I missing something.
console debug stmt.
De obicei in fisierele de config se pot specifica sau nu ghilimele. Hai sa nu folosim ca sa ne fie mai usor.
I don't think so, but there's no chance these differ by case is there?
Hey now, woah!

PHP itself is pretty quick, the slow parts are various bottlenecks come from interacting with the file system, making calls to MySQL, remote connections, etc. you may well go on to explain this later on but this is a terrible way to start things off!

> PHP is pretty quick by itself, but bottlenecks can arise when you make remote connections, load files, etc. There are various tools available to speed up certain parts of your application, or reduce the number of times these various time consuming tasks need to run.
Надо нормально парсить целиком число, а не по одной цифре.
Иначе, получается class java.lang.reflect.InvocationTargetException Message:
Thanks Niklas!
Maybe call this options "tooltipContentFunction", since it's in the defaults block we don't need to repeat 'default' in the name.
We normally say "deployed to" not "deployed into".
Actually probably not.  Good catch.
this is a base page... so different import error may appear when using this methods in tests
Have you tried calling this as soon as the Persona popup window opens and WebDriver has switched to it? (but before the login starts)
You are right, it will be deleted.
@ahojnnes Okay!  Thank you.  Will follow the development of `corner_peaks`.
Ok then let's just delete those lines for now and we will see later if we actually need this.
ever-increasing
On Wed, Nov 07, 2012 at 09:19:32AM -0800, Andreas Mueller wrote:
> how? overwrite the init paramter?

Indeed, it does pose a problem.

I am not sure what to do. I would way: let us break compatibility here:
HMMs are not used a lot yet.
I would use the tools from `sklearn.preprocessing` here. Or maybe even a whitening PCA, I remember hearing it works well for this.
yeah it gets a little complicated since node pre-allocates bigger chunk etc internally so it's not such a clear 1-1 relationship with regular allocations. What I did when I attempted to decrease the number of `.write()`s by concatenating the Batch buffers was that it  ended up being much much slower for larger messages, but good for very tiny ones (which are likely unrealistic)
Currently fails, should probably removed to match the new ARIA spec.
It's like that for weather, too. I originally had it like that, @W1ndwaker told me to. :P

I must have forgotten to change it to weather when I copy+pasted it from the time command. :P
I know this is correct according to PEP8, but I don't like that part of the PEP8 style. Given that this is in the jukebox, which is a bastard's child anyway, I'll let it pass ;-)
Arrête une tache et retourne son statut
This looks like a copy-paste bug.  Is it really supposed to be called "depot_form" or was that an example from the book?
We should still suggest the injection way. It is the most powerful feature suggested by the module. Maybe we should just remove the controller example (em in controller is surely not something to suggest).
here it is to avoid importing the tika classes. If I call directly a method on _org.hibernate.search.bridge.builtin.TikaBridge_ I start pulling in the whole TIka stuff. _BridgeFactory_ is the point where "Tika" is made optional.
I did wonder...
the else code (line 2322) is 1st initialization of min size, so no need to shrink there.

But here, it could be big, I don't know whether worth it, or let it be and stay large, + let critical push it out if need be..
Not sure..
Yes, indeed. I'll change that.
Тоже я бы не стал @ возле ```tag``` рисовать, возможно это только моё бессмысленное мнение :smile:
This is raising the following exception:

    [Errno 20] Not a directory: '/var/folders/5_/9gz0dv9s51q2shxc8hg62j200000gn/T/selenium-ide-editor-1.9.0.xpi/install.rdf'

I suspect that the XPI needs to be unzipped.
An implementation of "PuppetOptionParser" based on the ruby stdlib OptionParser.  This is not currently in use because there are a couple of specs that it doesn't pass, but it's here in case we find faults in trollop and feel that we need to revert back to this.
I'd argue we should change the behavior of the JsonModel to not have a captureTo default. This would accomplish what you're looking for, and not require additional flags.
Here, add this:

```ruby
:not_if node["keystone"]["signing"]["token_format"] == "UUID"
```

Also, from what I can tell, keystone-manage pki_setup should only be run for self-signed certs. Is that true?
place sendkey in try catch
Tässä voisit hyödyntää `rank` funktiota seuraavasti

~~~clojure
(defn rank-ace-low [card]
  (replace {14 1} (rank card)))
~~~
Oh, I misunderstood what you meant by the comment; I thought that the URL wasn't working anymore.
Just delete, we have git to keep old code available.
Lets discuss, I left it vague on purpose, but I can change




On 13 בדצמ 2012, at 23:35, Ran Tavory <notifications@github.com> wrote:

In defensive_programming.md:

> @@ -0,0 +1,17 @@
> +Title: Defensive Programming - I know what you did last sprint?
> +================================================================
> +
> +People have a lot of approaches when developing their own code, and when looking at existing code.
> +In this talk I will discuss I call "defensive programming" (not the [secure programming](http://en.wikipedia.org/wiki/Defensive_programming) you might know from other places),
> +and how I see it affecting your team and the strength of your code

I like the subject, but I think you should add a little more to the
description. You say what Defensive Programming is not, but you do not say
what Defensive Programming is. Or (alternatively) what problems is it
trying to solve.

—
Reply to this email directly or view it on
GitHub<https://github.com/rantav/reversim-summit-2013/pull/11/files#r2414000>.
It isn't correct as Author field
true, I guess it should be connected = true, then later he changes it to false if the connect attempt fails.
Pretty backwards if you ask me, but there's more than one way to skin the ole cat.  
In my mind, 'connected' should reflect the current state, which is false until the 'connect' command doesn't fail
```ruby
connected = false
#other code
begin
   connect
   connected = true
   rescue ::Rex::ConnectionRefused
      print_status("#{ip}:#{datastore['RPORT']} - connection refused")
      connected = false
   rescue
   # etc etc
end
```
mmmh.. are attributes case sensitive ?
I do have a test case with runAs=userId that does ends end with an empty #{user}
typo: `optaining` -> `obtaining`
I thought @stefanv 's original comment here used "LNICS" just as an abbreviation for explaining the formatting. Personally, I think writing out "von Lecture notes in computer science" would be better.
You should run `mvn xar:format` first, this will set up the right metadata.
``lastest`` ?
looks more like python this way. Just sayin'
Need to add a resize down test too. Since we should be able to do both.
How does the off-center frustum and infinite project matrix interact?  Do things work as expected when picking sensors in the distance (I think that is the only thing the infinite projection matrix is used for, but it is possible that it will be used for shadow volumes later).
Sorry, you're right.  Ignore my comment.
Good catch, Kevin is right, you should use `expect(tAngle).toEqualEpsilon(uAngle, CesiumMath.EPSILON1);` instead of what you have.  Same for everywhere else `equalsEpsilon` is used in this file.
Maybe we should put the box at altitude so people don't wonder why the box is under the Earth even though the depth test is enabled (because of the depth plane).  Ditto for the other Sandbox example.
Given that this is in TextureAtlas, and creating a texture requires cross-origin images, I see no use for a flag, since it won't work if it's turned off anyway.
Let's just default it to undefined, and then only create the logo canvas if it's not undefined.  Otherwise, we're doing extra to create a canvas, only to draw nothing on it.  `WebMapServiceImageryProvider` is a good model to imitate.
How do you feel about omitting unused settings?
Command works fine as is and would still work even with a trailing slash on the driverRunWorkingDir variable but I still plan to change to join in order to improve clarity and for the sake of consistency.
object is created.
Really dont like assert. Just return false so it falls to the NSLog loop.
This is only used for font sizing.
persistent is rather flexible in iOS. It is allowed to change anytime from the dialog's creation to when we actually dismiss the dialog. This means the developer can change its value after show() is called.
По возможности используй относительный селекторы с `.find()` дело в том, что в таком случае область поиска ограничивается и происходит быстрее - ищется только в subtree.
Please use spaces around operators.
`deepTestBuild`? Certainly better than `a`. But, I had hard time thinking of a name for it...
I can make these changes if you want, but honestly, I'm working on proposal which would completely replace `dummyRanges`, so I'm not all that inclined to make lots of little tweaks to it.
Whats with the weird restructuring of imports?
yeah, everything is singular now, *nix-style
s/successful/successfully/
Someone's going to file a bug on this some time, I know it.
Fair enough
On Oct 8, 2012 10:11 AM, "Felix Mayerhuber" <notifications@github.com>
wrote:

> In
> components/ekb/common/src/main/java/org/openengsb/core/ekb/common/EDBConverterUtils.java:
>
> > @@ -30,6 +39,7 @@
> >   */
> >  public final class EDBConverterUtils {
> >      public static final String FILEWRAPPER_FILENAME_SUFFIX = ".filename";
> > +    public static final String REFERENCE_PREFIX = "refersTo_";
> >
> >      private EDBConverterUtils() {
>
> ok, I will think about that. But is it ok for you to make that also in an
> own issue with the goal to minimize this converter utils? I think that
> would be easier for me to program and for you to review ^^
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openengsb/openengsb-framework/pull/776/files#r1782561>.
>
>
I think that's just because this branch is now behind. There are no commits on it explicitly changing the bioformats submodule pointer at all. The branch just needs to have develop merged in again which I've now done. *shrugs*
Please move this line so that the URL appears in alphabetical order.
@orfin it means that all the classes you want to use this way need to extend DTO. What if you want to use a third party class as parent ? you can't.

As it is empty, using an interface would be better
Like Jens, mentioned, this should be included, it's available already, so comment this out.
We should restrict this selector, and the others similar to this, to just the download builder form, as the form will be embedded an another page later.
Thanks for pointing that out!  I updated the remote branch with the fix you suggested.
@nicolaasmatthijs So what is the reason behind using this plug-in (which exposes a global variable) and not the one that I suggested (or even other ones)?
This should not have ORUR01 in the name, please update this and the ticket desc. (and any wiki page that has been started)
Yes; of course; agreed.

Maybe you could change the javadoc during the merge...

s/subclasses must not implement/subclasses will not be able to override/

or

s/subclasses must not implement this method/this method will be private/

??
Yeah, read the description, but the old proverbial picture (log example) is
worth a thousand words. So, if I merge this, r u PRing it to the original
repo, or is this the one we are using at Medidata?

On Thu, Oct 4, 2012 at 3:20 PM, Ethan <notifications@github.com> wrote:

> In lib/rack/active_record_status.rb:
>
> > @@ -20,7 +20,7 @@ def call(env)
> >      def get_status
> >        begin
> >          # Check that the application is connected to the database
> > -        ActiveRecord::Base.connection.select_all('select 1')
> > +        ActiveRecord::Base.connection.select_all('select "app_status"')
>
> like the description says, it helps with grep -ving the logs.
>
> currently this shows up in the logs:
>
> D, [2012-10-04T22:01:40.204925 #28750] DEBUG -- : (0.8ms) select 1
>
> with this change, this shows up in the logs:
>
> D, [2012-10-04T22:01:40.204925 #28750] DEBUG -- : (0.8ms) select
> "app_status"
>
> this change means that grep -v app_status will filter out the latter.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mdsol/rack-active_record_status/pull/1/files#r1767660>.
>
>
And yeah, I eventually want to get actual tests in here. I haven't written any either, but want to get them in for all my libraries.
I think we should provide a good-practice examples. And I believe this code will grow more and more. So this should not be one simple example, but showcase of everything. So things could be ugly. We can have then in documentation separate smaller things.
the curly brace should be on its own line for the method definition
So I tested it and it didnt blow up.. Maybe i was getting bit before I upgraded to 3.2.1 and it just stuck with me. I think have an issue with region selection and either set_variable_values or replace_or_add maybe thats what is fixed at some later release that doesn't have a CFEngine RPM supplied.

At any rate, I will switch it back to set_variable_values and just keep an eye out for whatever oddity I was having when I first started using this pattern.
But wouldn't it also be viable to use the inline-block nature of images to simply cram them next to each other and lose a lot of the css?

*(Disclaimer: I didn't read the current version of the text closely yet, so I'm probably missing several reasons why you did things this way.)*
Nice. I guess that it won't collide with the configuration for LXCs.
I just noticed that you already have a dependency on `actionpack`, which depends on `activesupport`, so it's safe to use `singleton_class` instead.
Thats why I changed it. Your original README says "$this->OAuth->Clients"  :)
move this logic to presenter method
*(mon fils ma bataille)*, c'est un peu bizarre. si tu connais la chanson et de quoi elle parle, ca sous-entend que Cyril va se barrer et que vous allez vous disputer la garde...
It's relevant because people who write responsive webapps are going to grate against keeping all of the logic on the backend. Medley isn't that restrictive why should the test be?
setNotification is not an appropriate name.

Also if you move the one line of notificationBuilder.build() from the below method to here, you dont need to pass around the builder

And then this method can return notification itself, and can be called getNotification
Хорошо, что со ссылкой, плохо, что уволил Андрея из Mail.Ru обратно в HH %)
Okay sounds good, thanks for the insight.
Code duplication. We might want to track `TiDimensionIsAutoFill` through a `BOOL` in the enclosing scope, and then use that value to go through to the following `if` check. This will avoid work and possible mistakes if we ever have to update `FILL` behavior (unlikely, but possible).
Could you fix this identation ? Seems to have an extra "space" character. thanks :)
@samnang why did you choose this over the change I suggested in the controller?
CACHE_MIDDLEWARE_ANONYMOUS_ONLY and CACHE_TIMEOUT were introduced witch Askbot. So, as we do not previously define cache policy for IP, I suggest to keep these settings.
Can we not get rid of the https URIs? (and following)
Ah now I get the purpose of the Resources-Parameter :) Will change the code tomorrow
My idea was to basically have just one transaction manger implementation per test project, at the given time, just like we have for containers. Therefore each concrete implementation can do it's own metadata discovery which is unique for itself.

If we really foresee a need to handle more than one transaction manager for tests then we might need to introduce some sort of assist. But the question is - does it really happen in the projects? Even if so then you can always split it to several integration project (matching somehow the conceptual level of the app I believe).
why have you updated the webplatform??
looks like a mistake to me
Add to class as public static method
Comment on all getter's and setters'
What's with this "#" thing? ... What does that mean?
you can retrieve some information from the ClassMetadataFactory