labofsahil/github-event-dataset-2012 huggingface dataset:
der typkann im grunde gar nix. ist ne pissnormale tc, aber iwie mögen die ihn so sehr :P
Comment created by PyGithub
Sure :) Thanks!
minha mãe não deixa falar palavrão :P
s/b64encode_w/b64decode_w/
Wow, That's Fantastic!
'contributing'
From Reddit? :)

Please feel free to make a pull request and make these changes if you would like.  Otherwise, I will look into it when I get a chance :)

The main reason for me to put this stuff up on github is to get used to working with a team and integrating their code.

Either way, thanks for the comments!
Hi, is this change made on purpose? We have had problem where all but outermost p tags disappear when editing content. I did a test and problem is gone after reversing this change.
Types in showAnalysis are mixed up still. Referer and name/author is not displayed correctly.
I have corrected this local, should I commit the fix?

            $types = $this->type;//array_keys($this->type);
            $ratings = array();
            
            foreach($types as $type) {
                $rating = $this->classify($comment[$type], $type);
                $ratings[$type] = $rating;
            }
http://cdn.anonfiles.com/1329343954245.txt

Here is a logfile taken with LOGSOUND defined and the 'error' function implemented.
we should probably fix this in a better way. Like you said on the pull request, the errors should be handled by AXRWrapper
What does '3' mean here (I know it's not new code...)
This explains the "bug" we were seeing last time.
оно и так всё через Text_Parser прогоняется.
Yell random is 2
Step two you mean? It's a long way to go till all EAI is converted..
- Breaking from rest is OK if we need to do things besides CRUD.  (such as changing state from shelved to active)
 - /goals only needs to respond to json.  
 - I'm not familiar enough w/ backbone to be sure what it would take to change @url to /my_goals and expect updates to post to /goals.
Should AP_MODULE_MAGIC_AT_LEAST macro be used instead?

@pames -- I know you had concerns re: gateway before.  Does ap_.._ex() mitigate your concerns?
oh, you fixed... ok then
Thanks for making the contribution. I will be releasing rails_apps_composer v1.1.0 very soon.

-- Daniel
re-added "Jenovas " spell fix for rogues, with codestyle fixes.
I actually asked the same to chorny. While I do have commit access to the github repo, I do not have access to PAUSE for the same.
I changed this line: https://github.com/danielbachhuber/Edit-Flow/commit/4e440237f7a08e47f26a45a93211066885e503f0#L2R259 before pulling, so I had a merge conflict. I fixed the merge conflict and then committed.
Just came across this. Checking that the module is enabled is not strictly necessary -- it's just to avoid calling the filter_calendar_item_fields() method when the module is not enabled, right? I was just looking at [this task](https://www.pivotaltracker.com/story/show/23105495), and was considering whether that would be part of our snippet.
Le 12/02/2012 16:50, Puddi Puddi ! a écrit :
> CAD ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/OlivierB/Polyjoule_siteWeb/commit/96ef74087c4df2c4a51b64c940eafadbef915a39#commitcomment-957841
Utilise le style blue_tabular pour tes tableaux de gestion, le style 
formulaire pour tes formulaires, le style editor pour l'intégration des 
balises tinymce, le système de sélection multiple comme dans article et 
membre,.... Regarde dans ton fichier rubrique, je viens de l'adapter.
Any reason why the pprop and charset variables cannot be moved to inside the test on the next line? So:

```python
    if len(bad_chars) > 0:
        pprop = getToolByName(context, 'portal_properties')
        charset = getattr(pprop.site_properties, 'default_charset', 'utf-8')
        bad_chars = ''.join(bad_chars).decode(charset)
```

That way we save having to look up site properties and the character set until it is absolutely needed.
It looks to me like the spinlock is dropped in the body of the loop, and we call put without the lock held?

Also, I think this change would break when you have two processes calling fsync() on the directory... the first would claim the list and the second would happily succeed without waiting.
I did think about it some time ago about providing links to exploit source. But then I thought not to do so as it will create some overhead of maintaining the links to the source because the url for the exploit might change in the future.
This makes each_connected populate and array with the name of the connection as key instead of throwing the data directly into $post_>connected. This is useful to make multiple connection types each. I think this was present in earlier versions....
this is also handled earlier in the initscript
Example of documentation: Can `filename` be relative? Are there any security risks? What happens in case of error?
Redundant; Log.e will already print the trace.
Both of these headers are specified as seconds:

<pre>The value of this field can be either an HTTP-date or an integer number of seconds (in decimal) after the time of the response.</pre>

(Not that we handle the date format.)

Just use 3002.
This may no longer be necessary?  I think clang means we don't need this anymore (though we did when the doc on the intranet was created)
{icon size='L'}?
Should be in uploads, not on it's own mount
I fail at reverting.
I think this line caused me severe pain trying to rebuild WebConverger locally until I commented it out. live-build would complain that it couldn't fetch the Release file from the Debian mirror, blaming a caching proxy as the likely reason. Of course, I don't have a proxy running on localhost:3128 at all so it couldn't fetch anything... heh.
accepting 
@Subv  i sayed u u fucked it up aggain
Und auch hier ist "Anzuzeigende Links" die korrekte Form. (https://secure.wikimedia.org/wiktionary/de/wiki/Link)
I agree with you in the case of a registered menu, but when it is being dynamically assembled I think there are some cases where a developer does not want the menu to display. 

I had such a need. I tried just not setting the "items" but I ended up getting a 1 pixel high menu (no error was raised). I can explain my use case further if you want... Just not from my phone :)
C’est pas mis dans la CSS finale, c’est mis en valeur de l’attribut `style` de l’élément.

Et de manière générale: définir des styles en JavaScript c’est à éviter. Mieux vaut ajouter, supprimer ou modifier des classes prédéfinies, et gérer les styles pour les différents cas de figure dans ta feuille de styles. L’exception, c’est quand tu dois utiliser une valeur que tu ne connais pas à l’avance et que tu dois calculer en JavaScript. En dehors de ce cas de figure, les styles définis en JS c’est le mal.
change this into
return 0;
        //return get_allow_toggle_display() && (key_code == KEY_HOME || key_code == KEY_MENU || key_code == KEY_END);

remember to read default_recovery_ui.c on android_bootable_recovery

because every update will have different layout :)
See https://github.com/composer/composer/issues/419 - I tried to explain in details why it's like it is.
Good point, we want to say in the spec that parsing 1.0 descriptors is required. <https://hibernate.onjira.com/browse/BVAL-270>
Ok cool let me know and I'll update it.

We (myself gav and carey) were also talking about needing to update the overall layout because it doesn't make much sense any more.  It was fine when it was a poster for the first event but not now it's rolling updates with bits of promo material and stuff.  We can deal witht hat in another pull request I guess.
Hello.
"su" is a interactive shell command, hence you'd need to make something like a global living "shell" in the background to interact with it. I'd suggest you use the SuperUser app to grant it root access. This may require some more reworking of Gimite's original code to make it act like a shell.

Best
Will new lambda syntax be used in documentation?
`auto mfp = function(C self, int i) => self.foo(i);`
is a bit shorter than
`auto mfp = function(C self, int i) { return self.foo(i); };`

Looks like
`auto mfp = (C self, int i) => self.foo(i);`
can't be used because it doesn't say is it a `function` or a `delegate`. We can't use these keywords instead of `auto` so the shortest clear way
`function mfp = (C self, int i) => self.foo(i);`
is invalid now.
Carryover from Elasticache.  Fixing in both elasticache and rds.
we should not override create imho, rather then implement it in the save method.

if you look at the parent class, create is already defind as something like

  def create
    new
    save
  end
There are two different functions `setOption()`. One looks like a setter and other is a parser. Maybe the parser could go into the LisOption class constructor?
Ah, gotcha. thanks!
I think improving the search algorithm in a way that obviates the need for a hard comparison length is the way to go.
I tend to like pending or unfulfilled.
yes, in case the user uses one code for multiple kernels
facet_p undefined?
This is a problem now that `apply_config()` is not called from tronfig.  If tron gets this far and crashes it will bring down the entire service, which is not what we want.  I believe a solution would be to join names using  `.` instead of underscore. `.` is not a valid part of an identifier so it shouldn't be possible for any collisions.
Why is this needed at all? How does one discern a companion from the class?
needs discussion
Nope, there is not trying involved in either case. It’s a Resume, for two different reasons.
could this benefit from being lazy?
what combination was it that @jboner added runtime check on? should that be mentioned here?
there is a comment nine lines down which explains this
Would be easy to put a section in the camel config that maps the keys to FQCNs:

akka {
  camel {
    conversions {
       "file" = "java.io.InputStream",
       "pigdog" = "org.awesome.Pigdog",
       …
    }
  }
}


And then just load that from the config when the Extension is created.
I removed contiguous again.
remove the 2?
Mostly because I don't have much experience with multivariate minimization. I wanted to push out the work I've already done, but I'm happy to go back and optimize it, especially if I can get some help, but probably not until after the release.
Conceptually, there are plenty of other things we could do in here besides the call request archive stuff. We don't have to change it now, but it might make sense to move the history update into here as well.

Actually, maybe it does make sense. IIRC, there's a bug out there about the history state not being correct when cancelled. I don't know that we could know that where the history write currently is. But here I imagine it's pretty easy to analyze the call report and see if it was successful or cancelled.
I think we should shove the repo and distributor in here as well. If it was important before, it's important here. The "agent.bind" will indicate the step, but it's still needed to identify the requested operation itself.
The _href field needs to be added to this result before it is returned. This is actually the reason why execute_ok is deprecated.
setContextInMessage
@elupus: To be clear: It should become "URIUtils::ProtocolHasEncodedPath() then, right?
не, это безопасное использование, так и должно быть, почитай доку
сильно заморочено получилось. Тут же можно проще. Объявить константу GOOD_QUALITY_LABEL = _("Good"), ee заюзать в списке и вместо этого метода везде прописать StockItem.GOOD_QUALITY_LABEL
Does the RmPlugin, 'rm' command work here? Like in a JavaResource, you can 'rm myfield:String' :) Just an idea.
La longitud del nombre, debe ser de 200
It's better to do "assert page.has_css?(.'x-grid-row')" because then Capybara will poll for the grid to come up and be populated with data before it continues - the same for the above test
should it be like that ?
 #ifdef.s
  printk(".....");
 #endif
i don't think this needs filesystem either?
This actually should definitely not throw an exception.  It is a private method and it would be silly for the message class to have an entire exception that existed only so it could be caught before ever leaving the mesage class every time in a single private method.  Throwing an exception here would be a case of using exceptions for control flow - not desirable.
Couldn't you reuse the same interface for filters and functions instead ? They are the same.
Go to the [profiler] (https://thefiletree.com/profiler.html) and be horrified that days is an information shown twice.
This can come at a later date.
Oops, there are the same words below.
The exception will propagate upwards anyway. But I looked into Kryo source, and looks like input.close will also close the input stream. I am going to remove the second close from this.
Could you move the classpath munging here https://github.com/mirah/mirah/blob/master/lib/mirah.rb#L20
I've been consolidating the classpath changes in one place to make it easier to see which jars are added to the path.
According to the RFC, the only occurrence of time being sent to the client is via the /time command. All other time-y things are in "seconds elapsed". RPL_CREATIONTIME is non-standard, and therefore we have leniency on how to present it. UTC is probably the best course of action.

Side note: irc.topicAuthor sends a 333 numeric with a POSIX timestamp.
You can use `config.push()` here, like before your changes.
if (!(config && config.type)) {

would be safer here.
I suppose because the remainder of the functions in the openssl.c file contain similar methods of allocating memory for pointers prior to copying contents although most of them deal with the BIO_* family of functions provided by the ssl libs.

According to the documentation on the RETURN_STRINGL() macro it will actually perform the free/efree operations unless I am missing something
What do you mean by go upper?  What line are you referring to?
да, хочется знать, какая ошибка и почему она появлялась при сохранении? раньше оно при генерации кода создавало новый экземпляр репозитория, так что эту нужно было сохранять заранее. теперь оно использует этот же экземпляр репозитория?
Could you extract this condition to a method? Something like:

```ruby
if is_time(from_time) && is_time(to_time)
```
maybe `rounded_current_value` or something like this is a better name.
This line is still bad indented.
I think the `Example: ` could be in the same line.
Well, that proves that encodedName might not equal decodedName, but that wasn't the question.
While you've addressed backwards compatibility with the deprecated constructor and apply method, the change to `unapply` is not backwards compatible. 

An approach with an easier migration path might be:

```scala
object NoExternalID extends ExternalID {
  // ...
}
```

I was about to point out that this would also break existing code by adding a new subclass to a sealed hierarchy, but `ExternalID` isn't actually sealed.
While I like reduce I think it has gone out of fashion. I think the for loop is preferred style. I don't care too much about this (as I said I like reduce) but I thought I'd point it out once.
warpped -> wrapped

delete double space after comma in first line of docstring
That expression scares me. Must be ness.
@suralc würd ich direkt auch ändern
Settings here need to be of type string.  Boolean values are typically denoted with '0' and '1' in the configuration file.
Yes, please use double quotes. Consistency of quotes is desirable, but inconsistent quotes is preferable to unnecessary escaping.
Why do we need bigdecimal been added?
Sorry, I thought testing by flipping the method calls was sufficient but I see my mistake. Will fix and grunt in both versions.
removed in next commit a089006
This line is unnecessary with the below addition.
Sorry to be a pain, but can we make these values configurable as well? I like to make every possible value configurable :).
i checked with ircprotocolmanager and session... the lock is not used anywhere else, I don't see why I placed it here.
and, no comma.
May want to couch this a little.  There is _some_ interoperability, and more is coming.
indentation looks like it's 1 off?
That will cause issues. I think a second function that retrieves all drivers is the best way to go here.
Please re-remove this, it isn't related to the ticket.
Wouldn't using a limit be faster and use less memory?
`<!-- ENDIF -->>` ?
Maybe "Manage attachment extensions" and "Board extensions."
+1 on singular
Using an early return here may make the following else block a bit clearer (since the else starts on line 82 and ends on line 155).
Didn't catch this before merging and pushing, but we can use input type="number" to use a html5 number input (with validation and all) - would be even nicer if it had max set to the max sector id.
Two in here.
It's not half-bad either
zindex ro bardar, div.customer_title ro bezar bade custoner_scroll
I'm not sure about this. I like the idea of speeding up columns without NAs, of course, but not sure about removing the AbstractDataVec requirement. Things like the NA rules belong to the ADV (moot in this case, of course), and in the future I suspect indexes may be defined at that level too. I'd prefer something like CompleteDataVec <: AbstractDataVec that allows creation of methods that will work on any column.
Please avoid commenting out stuff, we have a SCM for viewing differences between changes :-)
Indentation should line up with '('
This appears redundant as target_coords only contains DimCoords, although I'm not convinced it should (see above).
n.b. need to remove the comment that the coordinate "cannot currently be bounded".
please use ```numpy.linspace``` if your going to do this. 
My preference would be to use the stock cubes - could you use one and just delete the CS?
This is a nice way of testing the GribWrapper without needing lots of test files. Definitely something to take forward.
drop m_partitionIndex?
m_partitionIndex seems to have 2 (ab)uses, neither of which really justify it.
Here it is used to pass the parameter index back to the caller of this void function (like the return value it had always -- er, sometimes -- wanted).
-- IF the caller happens to be QueryPlanner.plan, it pounces on the value immediately to initialize the CompiledPlan member -- end of usefulness. The other caller ignores m_partitionIndex completely.

The other (ab)use of m_partitionIndex is in the alternate QueryPlanner.plan code path.
There, m_partitionIndex donates its unused default -1 value to the less fortunate CompiledPlan member that was born without one, burdened from the start with an ambiguous 0. That's curable.  If it's even significant on that code path. Better safe than never.
Those do nothing, you don't need them.
mass deployment
jQuery решил не брать с CDN?
D3 тоже можно брать с CDN:
http://d3js.org/d3.v2.js
gcc 4.7.2 does not generate any warning without this useless assignment."

gui.c: In function ‘clean_modifier_combo_box’:
gui.c:730:31: warning: ignoring return value of ‘g_list_remove_link’, declared
with attribute warn_unused_result [-Wunused-result]

I'm running Ubuntu 12.4.

tlaberge@tdf:() -> gcc -v
gcc version 4.6.3 (Ubuntu/Linaro 4.6.3-1ubuntu5)

Perhaps this is fixed in later versions of gcc.
Because to me this is easier to read the way it is written! :) But, this is second opinion asking a change to this. So, will change.
Can we be sure of the error code here?
I really think maybe this error name should be a bit clearer, but that can be fixed later.
I think that we should do what maximizes developer happiness here. One of the stated goals of this rewrite was to enable a dev to drop in another timer or hook easily, so if necessitating a prefilter to provide your own timer is unnecessarily cumbersome, I'm against it. Do we have an example of how someone time do that?
Comment to explain these variables is needed.
Is there a reason you prefer array constructors over array_agg?  I don't think it makes much difference, just curious.
Don't you need to DROP tr_association when sampling_needed = true?
I've just had a quick look at some other plugins and, for example, the PostGIS plugin also creates one Context for each call to features() like the PythonDatasource.wkb_features() implementation does.
s/unicode/bytes/?
Why not change the `>>` to `>` and remove the touch?
I would use

```rb
branch = `git symbolic-ref --short -q HEAD`.chomp
```
Relatively. I have an update with the 0.5.0 release that I haven't pushed yet.
Good point. I don't know. Probably safer to have the dev tools upfront!
Look at the `f()` format function for an easier way to deal with string concatenation :)
Well, if Pymacs was any sizable, yes.  With only two files, it's a tiny project, not worth much structure!
We're no longer returning the message now. This is problematic for `send_to()`.
Extra 10 lines.
Yeah, but it's still a bug. This a cut and paste error!
Sorry - missed a bunch of callers.

BTW, this is why I wanted to use Truth.

ASSERT.that(ep.strings).has().exactly("string1", "string2");

Not advocating... just sayin'.  :)
Here it is a bit tricky depending on the operator precedence (which I don't remember) if it is:
 ``values[i] = (rand() % highest_value) + 1;`` then I guess we are ok, but if it is:
 ``values[i] = rand() % (highest_value + 1);`` then we probably have a problem (try to print the value ``uint8_t value = UINT8_MAX + 1;``)
Ugh!! Habits are hard to change. I will amend it.
I'd suggest using exit 1 for consistency here
I'll fix this for all Vaults as Its implemented relatively the same.
`> 0` is implicit and doesn't really add any readability
The name is so long that it actually takes more time to figure out what constant is being used, and object property names don't get minified. I feel a bikeshed coming on so if you think it is better than its cool but I find it more confusing to be honest.
may as well combine these into `margin: 1px -10px 0 -10px`
This seems too dark and too harsh of a dividing line. Compare it to the bottom border of the image gallery region and it's much darker.
why `adapter = window`?
You are right. I removed rollout/rollover. Those events were intended for Menu (so that clients could update the menu item appearance when a menus is shown) and they events were accidentally copied to MenuItem.

I will remove hide/show as well. Those were poorly named. My intent was more focusIn and focusOut so clients could do something on rollover, but I don't think this is a common use case, so I will remove it. We can add later if there is a reason.
複数の拡張子の表現が使われているパターンに対応
I see `ace/mode/css/csslint` is pretty old
Whenever you update it and we settle to use an ace with the updated `csslint` -- we may remove it seamlessly
This is the only thing I would change,  I see your point that having that many queue'd request would be a "bad thing", but it may also be hard to debug if we are not parsing the right value here (but it's way too late for me to even follow what would happen)

That being said, my mac is back, so I should be able to try this branch on my work app and another app I have.
this file is rename by cmake. see $1
Forgot to delete it. It's auto create by pycharm.
Same as bellow. GET requests shouldn't change anything.
Let's just make this a const up at the top.
Same with this three, should be numbers I think.
just add  `[ "transition" ]` here
Both are actually valid. This is straight out of jquery UI... weird that they would use different syntax, but I don't want to go messing with it now
I actually really don't mind this being `(function(){` because the space is just... entirely unnecessary.
Can you do this without using classList?
All of the changes to how we apply/interact with `data-*` attributes look like they have the reverse change applied to them. They *should* all be `data-*`, and we had a ticket to change them to be appropriate. This oddity caught my eye in the CSS change above too -- the same switch happened. Perhaps this change didn't get passed back to 0.5.x properly.
We should file a bug on this so it doesn't get forgotten and then mention the number on this line
Ha - that's the default of my text editor. It will be gone
I could have sworn I fixed that...
A missing header...
或者直接使用 name
Let's remove that before merging if possible.
clean up whitespace of this conditional and remove semilcolons - thanks!
Unprintable character here
you should be able to share the 1286/7 pin mapping.  Also put in 647 while you are at it.
Could this tag type be 'i', recent changes are pushing icons to be defined with an 'i' tag as they are easier to know the intended meaning and the HTML semantic definition for that tag fits the intent of icons best of the various tags.
1) Since you ve skipped auth,  we cannot be sure if the "current_user" is set or not..
2) Also Organization.any_readable? is preferable than to hard code rules logic in the view.

basically I am saying try  ->  if current_user && Organization.any_readable? i
Again, I have missed that in the initial review. The thing is our SELinux policy is currently very permissive and allowing execution of subprocesses, but we are likely to harden it and this will need to go away. Just a heads up - we do not want to spawn processes in the apache (or thin) process.

Let's leave it there until we hit this when working on SELinux again - then we will need to take a different approach if we want to show this.
this line seems odd. maybe I'm missing part of the code or something?
I think the Pythonic way here would be `if env is None`
Does it need quotes at all?







On Wed, Nov 21, 2012, at 11:01 AM, Aaron Steele wrote:

  In src/clj/fossa/utils.clj:
> @@ -263,3 +263,16 @@
>    (let [s (str "INSERT INTO gbif_points (name, occids, precision, year, month
,"
>                 " season, the_geom_multipoint) values (%s, ST_GeomFromText('%s
', 4326))")]
>      (format s value-str multi-point)))
> +
> +(defn mk-update-stmt
> +  "Create update statement given a value string and multi-point
> +  string"
> +  [sci-name field-name value-str]
> +  (let [table "gbif_points"
> +        s (str "UPDATE %s SET %s = %s WHERE \"name\" = '%s';")]

  Yeah kill the double quotes around name, it's not needed.

  —

Reply to this email directly or [1]view it on GitHub.

  [Jshd8sI44GVrKZBvymxqKBzA1W2W8nIp_4yHeDlxezPyKdXjP9ZeHwVRUpc6zR4p.gi
  f]

References

1. https://github.com/MapofLife/fossa/pull/31/files#r2197842
Zašto si ovo maknuo ipak?
performance: maybe move the queue init and destruction out of the loop and just queue_empty it before each pass.
@cgjones can you describe what would happen if you didn't skip these messages?
哥，这东西不靠谱。

如果我设置每9分钟一次，你给我 9 18 27 36 54 运行，那54~9 不是直接等了15分钟。

而且我设置了9:15 开始， 看起来8:15的时候就会跑了。 这东西合逻辑么？
https://github.com/Weltschmerz/Haraka/blob/a1fab6a19035c2f60835d7edee468e3df619d12a/outbound.js#L8
pWalletMap can be allocated statically, I believe.
En français (désolé). Il y avait en effet incompréhension.
L'issue mettait en évidence un problème de cohérence avec : 
 - dans un cas, désactivation de l'outil (résultats reçus),
 - dans un autre, outil qui reste actif (pas de résultats).

Il semble que le projet RegioGIS est le seul projet dans lequel cette notion de désactivation automatique avait lieu.
Il faut retrouver comment c'était fait pour ce projet. Et décider si cette désactivation est une fonctionnalité souhaitée.
*I am using emacs and bind 4 spaces for a tab.
*
On Thu, Aug 16, 2012 at 6:39 PM, Riccardo <notifications@github.com> wrote:

> In emesene/e3/webqq/Worker.py:
>
> > +        pass
> > +
> > +    def _handle_action_set_nick(self, nick):
> > +        '''handle Action.ACTION_SET_NICK
> > +        '''
> > +        pass
> > +
> > +    def _handle_action_set_picture(self, picture_name):
> > +        '''handle Action.ACTION_SET_PICTURE
> > +        '''
> > +        pass
> > +
> > +    def _handle_action_new_conversation(self, account, cid):
> > +        '''handle Action.ACTION_NEW_CONVERSATION
> > +        '''
> > +	print "New conversation account : %s, cid : %s" %(account, cid)
>
> i only see 2 spaces, maybe they're tabs?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/emesene/emesene/pull/1460/files#r1391353>.
>
>



-- 

*Best Regards~*


*Xiang Wang*

*CDC, TrendMicro*
You could use com.google.common.collect.Lists instead.
So this should be parsed? I think you are creating invalid jCal with this, a float value shold always be a number.
A loop will make your life wonderful. Haven't tested this, but something similar should work if this doesn't:

```jinja
{% for i in range(1, 67) %}
  <li><a><img src="{{ media('img/mosaic/' ~ '%03d'|format(i) ~ '.jpg') }}" alt=""></a></li>
{% endfor %}
```
I'd vote for removing the ")" for these numbers. It doesn't look good in a heading and the dot is enough by itself. So just "1. Learn Why This Matters" and so on.
the reasoning here is that I'm taking two potentially incompatible scopes and putting them into one file.  I've isolated those potentially incompatible scopes into the two respective 2008 and 2012 functions.  This likely overly conservative, but I like having these imports in isolation.
Python's ``str(my_datetime)`` doesn't put a T here, so why should I? :)
missing line return
Stranger danger.
Maybe we could use the `Process` component here if available (and add it as a suggestion in composer.json). The `ExecutableFinder`, `Process` and `ProcessBuilder` could be used in this class and in the current GnuFindAdapter too.
What do you think?
the curly brace should be on its own line for methods and classes. See http://symfony.com/doc/current/contributing/code/standards.html
Added phpdoc descriptions for all significant classes
Makes it a bit more readable don't you think?
Yes, otherwise a simple input field cannot be submitted. If an empty collection form has a CSRF field, it doesn't hurt anyone.
Instead of adding `@api` tags you should add `@return` once here, and below.
`FormViewInterface` is not used
I'm not sure either. The reason I went with the to_s is just thinking it might be easier to catch if we change the name of the Group class. That's highly unlikely though.
If the nick being banned is invalid, the worst that'll happen is it takes up space in the q:line list.
More importantly, checking for valid nicks would require another VALID_NICKNAME that also allows * and ? to appear anywhere (maybe even with length check in case someone puts a ton of asterisks in a row).

I can do it as long as you don't mind a VALID_NICKMASK or such regex to match such a thing.
Break out the statistics into a separate structure since they are mutable. This allows us to pass the MaketxParams as const.
Larry did this, I was confused at first too. But it is so cool.

Ok, so at that point we know that a is invalid and b is valid. So we just need to copy b to r. We have two cases, B having alpha channel or not:

Every time I say "we know" it is because the bad cases were filtered in ImageBufAlgo::over, so we are left only with the good cases here and therefore can simplify many things.

1. B has alpha: if B has alpha then we know that it must match the alpha channel of R, so the for loop covers this case.
for (int c = 0; c < channels_B; c++) { r[c] = b[c]; }

2. B does not have alpha: this implies that B has either 3 or 4 channels, since we do not allow images with number of channels different than 3 and 4 not to have alpha channel specified.

The reason is, for 3 and 4 channels if you don't tell us alpha channel we can assume for you RGB and RGBA, but if you give us an image with 5, 6 or whatever channels, we can't assume anything reasonable so we fail those cases.

Ok, now that we know that B has either 3 or 4 channels lets look at both cases.

If it has 4 channels, that case was handled in the for loop because we know that R also must have 4 channels and it has alpha channel at index 3, and for B we also assume alpha with index 3, that is RGBA.

So the for loop does the whole job, we are only left now to deal with the case where B has 3 channels, no alpha channel, and we assume alpha = 1 everywhere. For that case we know that R has 4 channels and alpha channel has index 3, and that is why r[3] = 1.0.

How do we know all that again? Well, the bad cases are filtered in ImageBufAlgo::over and they don't get to over_RAB, and once we get here we know things such as: "If B does not have an alpha channel, then it must have 3 or 4 channels. We also know that R has 4 channels and alpha channel with index 3."
bash actually highlights these
    ```bash
This file should be smaller, or use html with style="width:80%" to it can shrink on smaller browsers.
in `manifest_put` for example, there should be a check like (pseudocode):

```python
d = json.parse(request.raw_post_data)
assert d.id == descriptor, "Tried to PUT data about %s to URL %s"%(d.id, descriptor)
```

... and so on for the other functions.
Hmm, I removed them because the method name says it all actually. Want me to put them back?
love this changes .... much better
Oh, no, not a good idea ;)
There is a TODO here, i dont know how Toolbox actually knows that one data entry is done, and another begins. I tried double spacing, but sometimes the editors notes have double spacing... so in this case, i knew that the verb lexicon started with \ge for each new datum I also tried an export from a toolbox sample of internlinear glossing, there each datum starts with \ref so this code wont work for all toolbox, only for the sample file that i was testing.
ah! i think i found something i forgot to test, this might be the source of 428 column headings, they might repeat! that would be awesome if that was the case. I will check this code and commit any changes.
This subsystem might be ignored on the slave, in which case we should not throw en error when sending the operation. Instead we should do this on return. Emanuel is working on some stuff to make that easier, so I say we hold off on this until that is in.
L134-148 can be replace with:
    // Construct a Date object with date time specified in a ISO 8601 string (YYYY-MM-DDTHH:MM)
    var time = new Date(gDatePicker.value + 'T' + gTimePicker.value);

If you can make sure `gDatePicker.value` and `gTimePicker.value` is the format you stated.
if (callbacks !== undefined)
 return;
You may want to use some local variable  |canvas|, |context|, |region|
The call back function is invoked by WifiManager when its state changed. When the state changes from "connected" to "disconnected", because Wifi is disabled by user, I don't it overwrites "disabled" message on UI.
it's not exactly the part of this pull request (it's last pull's feature), and afaik window scrolling is accelerated as well, but if you want I can try to implement it using CSS.
You probably want to leave this block (and below) alone.
No, waiting until `bluetooth.onadapteradded` is better, it's sort of an API design issue. :'(
`bluetooth.onenabled` would be triggered first, and then `bluetooth.onadapteradded`.
But we need the adapter ready then the rest of things could happen.
ok, although probably that would need to be revised throughout all the apps, so I will fix as suggested on this code, that will be easier ;)
oops I have just seen @davidflanagan comment, great!
Looks good, except the semi-colon and start and end brace for the else should be removed for a single line block.
Isn't it simpler to just use "while" here?
BTW : in that case would be better if we do sorting by area?
Michas volani se zavorkama a bez zavorek. Jsou k tomu nejake pravidla? Jakoze zavorky defaultne nepouzivame, pridame je jen, pokud by to bez nich nefungovalo, nebo tak neco?
Done, thank!
Also, [Pro Git](http://git-scm.com/book) is a great (and free!) book about Git.
It's awkward because `profile_examples` can be an integer or a boolean value.  From @dchelimsky, it sounds like leaving it is the best option.
おなじく <code>
i still need a way for alex'es site to render "Saturday Sept 29" to the users
Better `"""Should get an environment when given an existing ID"""`. And next you could add a test for `"""Should return 404 when given a non existing environment ID"""`
Actually, I'm going to ignore this one. Calling, type parameters, and function definitions are three different cases as far as I'm concerned. I like the space on the function definition because there is twice the data there (name and type). In the other two cases, there is only type OR name.
For this case, object just felt right, but here's a stack overflow on the subject (that also tends towards case object):

http://stackoverflow.com/questions/1898932/case-classes-vs-enumerations-in-scala
body p { margin-bottom: 0cm; margin-top: 0pt; } 
    On 28/5/2012 22:01, Argyris Zymnis wrote:
    
      
        @@ -186,11 +189,13 @@ abstract class FixedPathSource(path : String*) extends FileSource {
 /**
 * Tab separated value source
 */
-case class Tsv(p : String, f : Fields = Fields.ALL) extends FixedPathSource(p)
+
+case class Tsv(p : String, f : Fields = Fields.ALL, sh : Boolean = false, wh: Boolean = false) extends FixedPathSource(p)

      
      
Maybe rename sh to showHeaders and wh to writeHeaders?

---
Reply to this email directly or view it on GitHub:
https://github.com/twitter/scalding/pull/85/files#r889434

    
    I didn't want to break the coding style I found throughout the
      FileSource, that uses abbreviated params, such as p:String,
      f:Fields ... etc. Your call !
Please remove or use the "safe" logger WDN.log().
well, the thing is join accepts any iterator, so you can use a generator instead of a list. The following 3 expressions are equivalent (actually, the generator ones should even be more efficient):

    ','.join([l.alpha3 for l in languages])  # list
    ','.join((l.alpha3 for l in languages))  # generator
    ','.join(l.alpha3 for l in languages)    # same as before, nicer syntax
I somehow don't like this;

the initilize() function is too close to wickets 
@Override
    protected void onInitialize() {
        super.onInitialize();
    }

that sits in every component;
There's no good way to do so, unfortunately, within the assumptions we make about db and mq.
I am with @smoeding . It is a reference to http://en.wiktionary.org/wiki/bean_counter . Beancounter is the domain terminology people expect to see when working with OpenVZ
**getRequest()** 取得一个**Request**对象，这个对象中就有**Representation**。所以可以考虑这个接口不用变。
I prefer scripts to have minimal output when things are "ok", so I'd rather not write anything that is "expected", like in this case.
Using the interface Triple rather than the class TripleImpl in proxies
Please avoid underscores in ecmd commands.
K, thanks for the note
This should not be committed!
Yep fixed.
o certo dentro das funções test_* seria utilizar Asserts, não?
Is this a standard JBoss thing or a setting from a particular target environment?  We should not let details specific to JBoss leak out into our program in general.
Yeah probably not, I think it slipped in from bug 792636
As the designated end-user curmudgeon, this fells bad / strange / confusing to me. Has Crystal reviewed this?
On 07/03/2012 07:45 AM, jeads wrote:
> To test, I ran the `manage.py create_project` command and this did
> not break on non-objectstore datasets.  However, it does break the
> unit tests.  

I'm guessing this is because you added the columns manually rather than
recreating the table using the Django ORM's syncdb, and did not include
the NOT NULL clause for those fields (which means they now match what
the ORM would generate anyway, since you've added null=True to the field
definition!)

I added `null=True` to the field definition in the
> model.  I prefer `NULL` over setting the value of a `VARCHAR` to an
> empty string.  I think there is a clear semantic distinction between
> a `NULL` and an empty string and that is "put no data" vs "put empty
> data".  There are also some unexpected consequences in SQL when using
> empty strings in place of `NULL`.  COUNT will count empty strings but
> not `NULL`s, we can discuss more later.

No need, I think you're right.

Carl
This should probably be in the target's description attribute.
For the test starting on line 482 that is correct.  The range does match and it should - "now" is 3/31 (a Thursday) at 22:30.  So, it qualifies for both the Thursday match and also the range match and should therefore succeed.

For the test starting on line 488 a better description might be: "should not match when the range doesn't match even if day-of-week matches" (The time range starts an hour after "now" even though "now" is still Thursday).
That sir, is an [excellent idea](https://github.com/slindberg/puppet/commit/d894b871211384b4d2ea90074997b775450d8945)
I felt like this "exit" call was unnecessary and bad practice, but not entirely sure about repercussions...
Thanks for verifying.
I think this should be a 403
should we handle parse exceptions if string cannot be parsed to booleans, etc.  Not likely to be a problem, but we don't want to crash the module in case someone messes with config files.
In case of exception, use logger / WARNING to log, and use defaults.
CacheBuilder seems like a good fit, but querulous seems to currently depend on a old version of Guava (r09). I'd rather we upgrade and use that, but I prefer defer to a separate change. The current approach is no worse than what we already use to memoize connection pools.
I'd still use the relative path, otherwise this has to be updated each time...
The same must be applied too to the rest of files
Thanks for the suggestion; fixed now.
Unsure if this shouldn't be done through the Zend\DI.
No need for an else here. If the condition previous matches, it returns. I.e., do this:
```
if (count($messages)) {
    return $messages[0];
}
return false;
```
no space between method name and parentheses -- rinse and repeat everywhere ;)
That change is from Christian Folini's original patch so I can't comment on the change.  I can say, at least, that it was a change from 4k to 64k in the original patch.  4k does strike me as a bit small in any case, considering the messages involved, but 64k is overkill.  I'm happy to revert this to 16k; I don't think that will hurt anything.
Configuration.php
I'm confused. Why do you need to test the app.config file of a Test project?
All these Array functions can be stacked on the same way I suggested you could stack on String functions:
`for (var key in Array.prototype) {
  if (Array.prototype.hasOwnProperty(key) && !ArrayTome.prototype.hasOwnProperty(key)) {
    ArrayTome.prototype[key] = Array.prototype[key];
  }
} `
Fixed in upcoming PR
Yes, I know, but this event would be useful at the ORM layer, because we can access information about the entities which are going to be persisted / removed at that point, which is not true if this would be at the DBAL level (although it would be useful if DBAL had these events too).

While it's true that transactions can be started manually before the unit of work start it, I think it's useful to fire the event at this exact point, where all the changes are calculated, and all the previous events were fired (even the onFlush event), so we can access the entities that are about to be persisted / removed at that exact moment. If "onBeginTransaction" would be at the DBAL level, we couldn't be sure at which point the transaction was started. 

Moreover, extensions could benefit from these events anyway. For example, think about an extension to handle the uploading of files. If the transaction fails, we can listen to the onRollback event to remove the filesif something goes wrong. This is currently not possible without this event. And we would need the information about the entities involved to remove the correct files, which, again, needs to be at the ORM level.

Now I'm talking about this, I'm convinced we need to use the UoW as another argument for the onBeginTransaction and onCommit events. And it would be even useful at the onRollback event, since that's the way I could access, for example, entities with their files, so I can remove them if an exception occurs.
I've just reverted access modifier but still there's the problem with access to file's resource properties (like mime type)
:vartype email: str right? There's a few more of those below
This isn't the most efficient way to do this. I'm just gonna cancel this pull request. Just file an issue and I'll fix it within the hour.
Underscore has a proper `_.isObject`.
We don't want to expand out JBoss AS 7 I think - JBoss AS 7 is its brand, just like JBoss Enterprise Application Platform is its.
Not sure if you need to detect both. Isn't the warning clear enough if you say "USE_I18N or USE_L10N is false"?
The addition of the trailing slash shouldn't be necessary, based on my testing:

[10:39:34.586] GET http://www-dev.allizom.org/firefox/mobile [HTTP/1.1 301 Moved Permanently 54ms]
[10:39:34.666] GET http://www-dev.allizom.org/mobile [HTTP/1.1 301 Moved Permanently 82ms]
I thought it was a thing one did for checkboxes
Oh, I missed this, thanks for pointing this out :)
The game got updated so I'm closing the pull request...
Should we just remove this completely?
Gotcha.  Sounds good.  I had implemented this using an example from https://www.quantopian.com, and should have translated better.
I wasn't sure of the implications of changing the default to false, so it's only being set locally.
Add a comment that mongodb or redis are loaded below based on configuration. This way we have all the required's listed in one place.
I wonder if we should make a helper on context of fileUtil that manages the bits of "if a file reference load, otherwise treat as a template". Could have other use cases.
I think I got it, and it looks good. But this would benefit from a method-level comment on the algorithm used, like: Check if my number is the lowest, if it is I have the lock, if not watch the lowest number and then try again within timeout.
when i test this I get:
Notice: Undefined index: config in formClass->createAdminForm() (line 322 of /mnt/share/drupal6/sites/default/modules/islandora/formClass.inc).

Can we test before we assign here?
Actually it's used in one component and more composites. I included component _footer.scss into _login.scss composite. This is the best solution for current situation, I think.
@ehelms  Actually, it imports the partial _forms.scss (the main one), which imports the rest of the files in folder.
yup.. maybe some day we can get rid of getVAR
Or should I use a regular expression to pull out just the number?
Ok, added it 2 lines below
Can you put it in the method doc block? Thx
I think that this can be made shorter:
"#page > .primary > h1"
I would put this function inside this one (as an inner function) rather than putting it at class level (unless I'm missing a use case where it could be useful to have the `_is_issuer_valid` method?)
Adding the soft switches as a memory callback here
/opt/puppet/bin only exists on PE installs.

![Alt text](http://www.unicorntears.com/images/unicry.jpg "Unicorn cries for us all")
We already talked about this in person, but for the sake of documenting our decisions:

The reason this lives externally is mostly because comparing versions is hard. :) But also, we may hypothetically change our version format or rules, in which case we can easily account for that on the server side. Similarly, we could add rules such as beta releases are the latest version, but aren't "newer" for the sake of whether or not to suggest an update, etc.
The boring stuff: docstring, preconditions, all that jazz.

Maybe call it `ensure-certname-is-active!`, or something? Man, I also suck at names. :(
I think you can probably move it anyway :)
There are a couple of places in this PR where you're passing `new Error(` rather than the `{'code': 500, 'msg': ..`
This JSDoc should say something about the expected format of configValues and should probably have an example.
Please change this as well.
A couple of methods in this class and `SocketAttachConnectorScala` are very similar. Could they be extracted in a base implementation class?
why is this not in the slanger namespace?
`else` needs to be on the next line, or you could just make the whole thing like this:

    return ($type !== 'array') ? $this->_fetch_object() : $this->_fetch_assoc();
Why not simply use :

    return (is_bool($str)) ? var_export($str, TRUE) : parent::escape($str);
Замечание: почему количество ячеек считаете по формуле,  а здесь используете константу? может должно быть self.DNA.count?
У нас массив из 100. Проценты равны единице.
Почему Вы здесь не используете dnaSize? Ведь если длина массива будет длинее 100, то алгоритм будет заменять только 100 первых позиций.