labofsahil/github-event-dataset-2012 huggingface dataset:
Added extra notes regarding how the messaging API is used
A URL with a dot preceding the question mark that leads the query params

...is extracted incorrectly. eg. http://t.co/a.?q=x
Ещё один фикс loadThread

Исправляет проблемы с реф-картой при подзагрузке поста.
Added the OpenCityCamp event in Ulm

Just a basic description.
working file saves

folder still has to be created tho, will work on that and then the tileset
Viewdraw parser tests

Poor thing looked all lonely without tests. Basic functionality is covered, but I'm not sure I like the structure of the whole thing, tbh.
Okay, now almost all of the changes are included (notes later).

Changed some height calculations to use this.element.dom.clientHeight,
as this.getHeight() wasn't always returning a non-zero answer.
Fixed bug where calling store.removeAll() or removing the last item in a
store would cause the list to be rendered incorrectly.
Fixed showEmptyText.
Rewrote the grouping code to address some problems.  First, the way the
grouping was working you had to group things by the same grouping as the
letters on the Index bar.  My rewrite makes a separation between the
mapping for the list groups and the mapping for the index bar.  It also
doesn't convert to lowercase but instead uses uppercase like the
indexbar does already.
I also made it where when you jump to a letter that's not represented in
the data, it jumps to the last record in the previous group.  It
basically jumps to where you WOULD be, if it had a list item with the
letter you tapped.
izbrisana stara koda, dodana nova

zbrisou sm uso staro kodo in paste-ou najnovejšo
Project Names can have spaces
updated to work with api change

Joey,

Here is the pull request. Sorry for the dealy.

Cheers,
Prabode
Moving externs into SFEMP3Shield class

I have no way to test it, but I moved the externs into the class as static members.  The C++ syntax is a little weird, but it compiles just fine, so it should run?
I18N and french L10N

tested with built-in 1.424 and manually on 1.473...
MAJ et trad

J'ai été un peu occupé ces derniers jours, mais voilà fini le "forestabandonedcampday" et une MAJ de "aliased_english"
mes corrections

fichier corrigé et terminé
Longer term we may be able to clean it up by:

a) Having the valueFn executed lazily, when the actual call to get/set happens [ maybe some subtle backwards compat issues here (the state of the instance maybe different when the valueFn is called lazily - but maybe something we can try out 3.6.0 PR1 ]

b) Having value of a subclass override valueFn of a superclass. Currently value/valueFn get merged across the hierarchy and then if there are both a valueFn and value at the end of that merge, valueFn takes precedence. The reason for this was that we wanted to come out of the aggregation process with a normalized name:value, instead of holding name:value, and a name:valueFn individually across the hierarchy (everything else gets merged). However we could get a performance improvement by delaying the valueFn call.
Also, now 2.5 is available. Welcome to travis!
Please add `const` to the getters.
Permutations like this (between Ruby and Node.js versions) are not supported. Please specify what `languages:` is the primary one and pick either `rvm` or `node_js`, respectively. Ruby VMs have Node.js 0.4, Node VMs have Ruby 1.9.2.
I'd prefer not to make static calls where we can avoid them, to ease php4/php5 compatibility.
n = 0 is outside the else block
Can't wait to read and talk about it, 've got plenty of stuff to publish (ideas uh, no written stuff yet).
But page cache is not to often operation (because it is cache) and compression is very fast operation.
SEE #4
Yes, that's the way Ogre work. I don't agree with their use of singletons; in Ember I want each object to have a clear owner.
I'm not saying that all Ember code is consistent. :)
This is something where you'll find older code which not always conform to what I've written above. As I alter the code I tend to try to update it to fit with the development practices I've settled on.
The main place where Ogre::Root is also used is as a field is in OgreSetup. In all other places we actually access it through the Singleton. This is ok since this is how Ogre is designed to work.
Those matches you've found are for other things which have mRoot in their names.
cc: @tenderlove 

Any reason this is not merged? Without this, no warning will be raised when a log directory isn't present
I also added some CSS classes to the code.
The kernel has a --jsonp option. (And the pid entry is used in reference to the --pid option, and when discussing init scripts.)
oh, convenient!
The 2 problems are now fixed in staging.
I'll set it to == 3.1 then. I'm not familiar enough with the code in question to know if it'll work without patching in 3.2. /cc @timtr
as $db is used only once this can go all on one line
after this commit world oPvP stopped working at all (at least in my repick).
This is wrong and it can't work, you want to do that at runtime, probably in a C function
Looks good, merging.
why did i add that extra whitespace? i shouldn't write code before having coffee.
So just remove the closure and expose the variables defined within? Or move very variable into the function (which would result in a too long function, IMHO).
I think we should delete the socket here as well. Close won't delete it if socket is not open. I just ran into this issue with vnsi.
Wouldn't be able to get any encoder data without these two lines.
This jacks up the minion on Windows.
I dont think that "pid *command*" is accurate. Maybe just say "should return Process.pid"
Didn't add my changes, this is the preloader.
CLI can make same guesses it was doing before. But it can just add a way for user to override them. Overriding may happen with the option like `--treat-as-c=C'. Or simple ini file in user's home directory. GUI then has its own ways to handle and select file extensions.
why close the instance s 8P fine on windows as far as i can tell 8)
да я не обижаюсь) мне пофиг если оно реально мешает можно было удалить .
лучше уж попробовать заменить на новую версию: http://sourceforge.net/projects/fastmm/files/FastMM%204.x%20Stable%20Releases/
ЗЫ а в чем ошибка в здесь поправленном запросе-то? в переносе строк? так это ошибка не в запросе, а в вашей софтине, которой вы обновы заливаете - вот ее и надо править. с комстроки заливается отлично.
ceil and floor - its very cool =)
good commit.
anal? ^^
Mathematically x / y = x * (1/y), but that's with infinite precision. With floating point the reciprocal is approximate, so the conversion can change the value of the result by a small amount. By default compilers try go get high precision results, unless you enable --fast-math or stuff like that
People were using location.href in quite a few spots and not taking into account URL parameters, port, etc.. Instead of copying/pasting these 4 lines in all those spots, i just abstracted it to a method.  That way people wouldn't have to think about URL parameters and I wouldn't have to search the code base for every spot using the current URL if query parameters or other URL modifications were introduced.  Its a valid consolidation of code both to eliminate redundancy and to aid in debugging...
In the future multiple commits for the same bug should be squashed (:
@garvinhicking ah, okay, now I understand, what it is: It's the token found in the email informing about new comments / trackbacks. You can moderate / delete directly via links found in that email. Thanks.
If these detection entries with incorrect filenames are kept for now, isn't this going to encourage users to rename their files and thus cause problems in the future if they get renamed back? Although if they don't *work* then disabling them solves that too.

Also: can't you just make the 'load room file' code modify the filename it tries to load, and not modify the data itself at all?
... which kind of supports my claim that #defining _UINT64 globally is a bad idea, doesn't it?
There's this which can be used for MSVC:
http://msinttypes.googlecode.com/svn/trunk/stdint.h

it looks as if it'll work in all of the MSVC versions... but the copyright notice on top looks a bit too much for a header file

Alternatively, we could only include this header file for the appropriate MSVC versions (i.e. if _MSC_VER >= 1000) and keep the defines for older versions?
Это должно быть у каждого класса, и время сна должно регулироваться через конструктор
Yeah I didn't know what I was doing... only looked up how to merge properly after I went through everything :(. Thanks for the help!
:p

On Thu, Mar 8, 2012 at 1:25 PM, Andreas Karfusehr <
reply@reply.github.com
> wrote:

> meow :D
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/naparuba/shinken/commit/a4a8e63f9750ac35f071e6050180dd97f994f8f8#commitcomment-1058151
>
translate string (noting this in several places. please ensure run all strings through translation engine.)
that's actually part of the reason that I double forked it (is that even a word)?

https://github.com/TheresNoBox/Hydrogen, although including the autoconfig as part of the project is certainly a better solution, then maintain two forks. I still might need to do that to avoid spilling application specific changes, but I think that branching might be able to cover my needs in that regard.  

Thanks!

BTW it's amazing what a little hope can do to one's desire to you know ... build things.
Well for instance, you made this in capitals :
 public enum Direction {
     NONE, EAST, WEST, SOUTH, NORTH 

And in my code I had :
dir = Direction.valueOf(arg.substring(2));

It stopped working after that. I had to put :
dir = Direction.valueOf(arg.substring(2).toUpperCase());

Also, I didn't like the change you did to the version#. It's again taken from the plugin.yml and again in format 0.1.1 instead of 0.1_1. I hate underscore.
Fixed in 4793097b89dab0e41c055e3683cb7562f8e58fed
Now I wonder if it is really necessary to do so much inreplacing in the makefiles. I assume going into the subdir and then calling make does not work?  (I have not read their README, nor have I ever compiled this)

Have you also checked that the dependencies are stated with "depends_on"?

From their website:

````
    Use this installer if you want to build Ice from sources. It includes header files, executables, debug/release DLLs, and JAR files for Ice's third-party dependencies:
    Berkeley DB 4.8.30
    OpenSSL 0.9.8r
    Libbzip2 1.0.6
    Expat 2.0.1
    mcpp 2.7.2
    JGoodies Common 1.2.0
    JGoodies Forms 1.4.1
    JGoodies Looks 2.4.1
    STLport 4.6.2
    Qt 4.5.3
````
Is there much overhead with verifying whether it is compressed or not?
is this required to be `https://`?
т.е. надо ждать продолжения где будет смена содержимого name? Сейчас там русские названия
you don't want to follow the `POST`. you want to `POST` and it normally

    r = self.client.post(url, self.data`)

and then

    self.assertRedirects(r, edit_url, 302)

would check that the redirect Location is set to the `edit_url`.
@philsturgeon ok, seems to have reset ok...do I need to sent another pull request upstream?
Remove these comments?
Right. ie. Tempate '%s' doesn't derive any resource.  Doesn't a primitive reference a template through its "template" attribute? 

Which description sounds natural here? :-)
Why `env` instead of just `./configure`?
`new` is of type `DeviceType` while `addon.device_types` is of type `AddonDeviceType`, the two sets will never intersect - unless there is a way to `addons.device_types.device_types`, that property name is misleading
Ah yes. I meant to do that.

Basically, it's not dynamic data. We want to be able to display different speech types in different ways, and this lets us avoid having code that depends on specific data in the database.

As it turns out, the way that I've ended up implementing it could equally have been done by poking an extra field into the database, but it still wouldn't really be dynamic data.
icanhasbukkit was showing up in the help by default because it aliases to a default command. It looked kind of silly. The example is still present for illustrative purposes - no knowledge was lost.
Existence of std::bitset - Section ends.
Might be too much code for too much status bits.
No I didn't. That's not a vendor prefix. ;)
Make this a normal variable on the class object, so you can `attr_reader` it.
So like.. I was just thinking Kendall.
How about using something like the code from python-vnc-viewer?
Efficient rendering inside the app itself allowing lifetime control.
Just a thought ^^ (Also, you might wanna think about heading over
to GiRepository code soon)
@simonster I was getting an error saying that the constant couldn't be redefined-- that makes me wonder if perhaps the sandbox wasn't being cleaned between running `detectImport` and `doImport`, but the code in Bookmarks.js was being executed twice. Probably not an issue, since most translators don't declare `const`s, but I don't think there should have been an error here.
Remove $data here too
верни всё взад... ты починил одно и сломал другое
should this be SETKEY/RMKEY, distinct from xattrs?  we should probably test both
this could be generalized to take [begin,end) arguments at some point
Brian - this is an interface, why does it need a new method?
Absolutely agree with @ikester. In my experience non-js json libraries tend to treat numbers as two distinct types. For example python's json library deserialize json numbers into ints and floats.
ппц =))) чота менда пуш болмаяпти(
ok i rebuild vmaps,maps and dbc with the latest extractor

now i get a memory error, if i start the core

TSCR: Loading Script Waypoints for 85 creature(s)...
>> Loaded 2222 Script Waypoint nodes in 10 ms

TSCR: Loading custom_npc_tele_category, custom_npc_tele_destination and custom_n                                                       pc_tele_association...

TSCR: >> Loaded 58 npc_teleport.
Loading C++ scripts
memory error
bummer city
wenn ich zuhause wieder inet hab schick ich dir mal nen link...
bzw schau mal auf wow-studio nach da gibtsn tutorial wie du ausm 4.3.2 nen 4.06 client machst.
SQL part is dedicated to all these people that pick things from random repos and don't say a shit nor give a fuck.

READ THE FUC*ING COMMIT

RETARDED Leacher!
want to make sure this doesn't throw an error when user incorrectly passes int (or other type)
the Blevins Rule -- after the third time you are allowed to consider it. That said, in this case if we did add the dep it would need to be shaded in.
I would really like to not add a dependency, particularly for two functions. If you copied the functions into the class instead (fine under AL2) I'd be happy to merge, though.
Use backticks instead `<code>` to show code. 

``    `test` ``
Fair enough -- I'll change it in a commit in a minute.
yeah this has no effect due to the later "set expandtab" line
This commit broke compilation for me on Windows as the dependency checking within qgroundcontrol.pro isn't correct. It's most likely due to the code around qgroundcontrol.pro:517.  Check my commit fixing this: Susurrus/qgroundcontrol:19805a92
`>= 0.95`
Wonderful use of fixed positioning! Great!
Hmm, one thing... incremental backup works by accident in that they will keep logical logs because the full backup kept one transaction in a log. Is that intentional to keep all logs for all incremental backups? I don't mind though, it's just that it takes more space, maybe lots of more space.
Such code should also check that the ranges aren't made to overlap?
Work fine, but character_aura need reset for fail save.
Si, muy interesantes tus comentarios. Te enseñare la forma que tú dices que deberia de ser, particularmente no me gusto mucho, pero en el próximo `commit` los hare notar.  Saludos
Thanks for the contribution.  I'll take a look at this hopefully soon so see if all is kosher
You don't need the pluses when you're wrapped in parens. Python automatically joins adjacent strings together like C.
Will do, left it in in case it was something you were keen to keep or translate later
あっ，ホントですね．修正します．ありがとうございますー
is this integrated in phpcr-odm? in DocumentManager, i only see createQuery which forwards to PHPCR

        $qm = $this->session->getWorkspace()->getQueryManager();
        return $qm->createQuery($statement, $language);

@nacmartin does it "by hand" in the sonata bundle
https://github.com/sonata-project/SonataDoctrinePhpcrAdminBundle/blob/master/Builder/DatagridBuilder.php#L130

i feel this should be in phpcr-odm DocumentManager
Thanks for the rapid answer.  I will do this.

Is the crash expected or should I send in a stack trace?

Best Regards,
John

On Feb 21, 2012, at 4:26 PM, Ken Collins wrote:

> I would have to do what you would do, just use some google-fu and find out. Once you find your answer, please share here and I can post to the README to help others.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/rails-sqlserver/tiny_tds/commit/c244f4faf24ca4e668371cbf9efaaff1dda78633#commitcomment-993857
...because the old macro used __attribute__ ((visibility ("default"))) which is the same for exporting and importing. Now with Q_DECL_EXPORT (__declspec(dllexport)) // Q_DECL_IMPORT (__declspec(dllimport)) importing/exporting is different.. the conclusion: it could never work in MSVC
Is that right? I thought the past tense of "cast" was "casted", but I don't know for sure.
s/clonse/clone
I would probably use FILTER_UNSAFE_RAW for passwords because we don't want to strip any characters from a user's password.
It would be useful to update the documentation to include best practices for this...
is there a reason for last super drop ? for me it is faster the old combo and more error prone (lift up super too soon)
This needs to happen in libwacom itself, not here.
c++ doesn't require variables to be declared at the beginning of a function, only C does.
It is misleading to have them at the beginning of a function in C++
declare i and j only when needed.
@diega: Thanks! I used your suggestion in my revisions for part 4.
vale pues cuando haya una version estable actualizo
_Ping?_
Well it there let me see it i can commit it
Somebody else posted a fixed compiled surfaceflinger on the forums, unknown source & patches used to fix it.
Beëindigen bij sluiten laatste tabblad
_print_comment is gone from TB2, that was likely the TB1 vs TB2 change which broke things.

Work at `done_testing` time is better done by adding an [EventHandler](http://search.cpan.org/perldoc?TB2::EventHandler) for the [TestEnd](http://search.cpan.org/perldoc?TB2::Event::TestEnd) event.  This will guarantee your module will work even with non Test::Builder based modules.

As for adding a message to the log role, I'm not sure why you're doing that.  You may simply want to make, issue and handle your own event.
Thoroughness appreciated. (But on the off chance you didn't know, you can abbreviate <tt>hg</tt> commands.)
Moin,

ich habe mal die Layouts weiter bearbeitet.
Hier der MagEnc komplett fertig als Bild im Anhang. Dafür musste ich 
einen Originalen mit Heisluft zerlegen.
Mich würde interessieren ob der MAgEncoder, weil er programierbar ist, 
eben auch programmiert ist.
Sonst muss ich da Reverse-Engineering betreiben.
Sonst läuft das mit dem Source-Code nicht, wenn da spezielle Parameter 
geproggt sind im MagEncoder.
Da die Motorendstufe schwer in dem Gehäuse zu bekommen ist, habe ich 
jetzt ein Angebot von China, für 50 Stück dieser Endstufen.
Da würde dann eine 6 Euro kosten. Da ich drignend 12Bit Servos brauche 
gehe ich diesen Weg, da die 10 Bit von Hitec diese auch nur auf 120 Grad 
haben.

Jetzt mache ich mich ans MCU-Board.

mfg

Borris


Jan Detlefsen schrieb:
> hey i'm german as well. lets keep this in english just in case anyone reads those (public) comments.
> yes, the idea is to collaborate on the board designs so everyone can make their own. makes more sense now that the project seems to be abandoned. I did is as far as i could by the material provided on their site but the designs are not complete. i would think 70% done maybe. also i'm a beginner with eagle so i'm not sure if those boards are technically working. i have never done designs and actually produced them. the idea was to start this project here on git and hopefully people with more skills would join.
>
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jandetlefsen/Supermodified-Servo/commit/9c188305fa147e924f23bbef3c60d8e97395dcb5#commitcomment-817426
>
>
For this particular situation, how about just adding a little paragraph to the docs explaining the relative benefits/pitfalls of putting the install args in each slot?  The example can't hope to communicate all that by itself.
it's not working when setted from Admin class now after last commits, working only from AdminController class

method Admin:setTemplates() is called few times and paste over $templates array with Configuration settings i think...
add chmod 0666 /dev/graphics/fb0 to this line for fb0 ( graphic ) permission
Ok i have one sugestion 

CMSG_REQUEST_ACCOUNT_DATA,   WorldSession::HandleUpdateAccountData
they are the same 
CMSG_UPDATE_ACCOUNT_DATA,       WorldSession::HandleUpdateAccountData
Victory! :-D

http://travis-ci.org/#!/DominikGuzei/ember-testing-on-rails/builds/678471
อาจารย์ทำยังไงต่อคะ
Not, not compatible.
Fare in modo che rientrino nel sistema di `add_to_form`, magari crando una funzione `move_field(form_class, field_name, fieldset_label)`
Comment disagree with code.
`else` on new line.
No need a second filter . The jQuery function outerWidth()  will  "Get the current computed width for the first element in the set of matched elements".   And all the pageIcons have  the same width.
use .ui-navbar instead
Using ID to find an element is not reliable. As there are two property views, only the button in the property view of layoutView will be found and it might is showing another widget's property if it's invisible.
There is already a ConfigTestCase in this file, I believe you overriding the first one here.
use single `"`
"cause" does not need to be a val. 
Why is this an Exception and not an AkkaException or similar?
`refs` is a field, `weights` is (hopefully) not, how about making one of them `private[this] val` and the other `_weights`?
type IndexedSeq[+A] = scala.collection.IndexedSeq[A]

So you get scala.collection.IndexedSeq, which isn't immutable :/
Don't capture "this" in your callbacks. Which you will do if you capture "channel".
I'm not keen about sending around closures either since it makes your program communicate using shared state.
The whole point of Routers is that normal actor performance does not suffice. In that case it will be beneficial to make the API user do the conversion instead of hiding it, so that the implications are completely clear.
Just a reminder to handle these codenames in a pack.  And the general issue of codenames declared in packs.  Remove on merge upstream.
Right, the problem is that none of this should be here.  Maybe if we fix something else we won't need the file anymore at all.  Otherwise we may have to generate it from the set of loaded modules (I don't like that either).

So, the whole file is a temporary thing, and it has to be there so we can load it for the tests.

And, I want to be able to put the 'sol' stuff in a place that allows us to run traits related tests for this part as an add-on op optional pack.

So, yes, all the sol stuff should be an a pack and not in any release unless 'optional'.
Didn't look like we use this.  It was unmask_text in most places.  We may use it for the new API once we know what we really want.
To be honest I think that there should be a `owner` field added to the dataset which would enforce this

the group_ownership would be like a mirror of that in practice
This makes me really happy that the itinerary approach removes this. I never liked the idea that the managers had this sort of complex logic.
should be after the log. I was wondering about this message: ParsePacket - {0} profile changed from 100 to 100
dc = pa.distribution_centres
Basically there is no problem. Only fields there are exists in ``$data```will be considered in output.
Is this called frequently? If so, you want to try to avoid closure-generating things like getOrElse
This is a variable that is set exactly once. Roland proposed a lazy val  
here, but then we don't have nice error handling. It deserves a comment  
though.
Ooh, that makes sense. Thanks!
Scratch that, building clean without that workaround seems to have cleared up the issue. I'll submit a new pull request.
This shouldn't be declared here. It should be in a private header specifically for `TUITableViewCell`.
SSL_SESS_CACHE_NO_AUTO_CLEAR?
`assert.deepEqual()`?
Use unsigned long, not off_t.
Right, exactly. It's in the <a href="https://github.com/amdjs/amdjs-api/wiki/AMD#wiki-define-dependencies">AMD spec</a> so it should be good across AMD loaders.
I don't like exposing these as public attributes; better to hide them via privileged methods.
Update to new class
php's  version  that you used is error  plesae use the last version. php 5.3.x  will cause  timezone dependent
а давайте этот метод в сцену вынесем, mainwindow ему не нужен вроде
@steveklabnik would it be kosher to plug http://github.com/bcardarella/client_side_validations in the guide?
Also I think that we don't need to delete the key. Only `options.fetch(:wrapper_tag, :p)` should be fine.
Moved to to and inside `ValidationsTest`. Making it inside looks a better idea to not make global namespace dirby and this is way it is done in other tests.
Agree -- no need to introduce a singleton for the custom case
I think "a selection of apply from something" is more or less generic. There are similar thingies in TreeInfo.
(And update commit message)
Okay, I'm sorry, you appear to have it right. I was glad to flush out the NO_NAME thing anyway.  I'll fix those and add the assert.
Note:  This one is my fault.  We just need something that captures Class VerifyError (and related) exceptions as well as ExceptionInInitializationErrors for static objects that bomb on classload.    Since this class is only used in our build, it's not critical for a release.
Compare with abstractTypeToBounds[1], which has several more cases. Can you elaborate on the difference? Intuitively, if one wants this case, one wants the rest of them.

[1] https://github.com/scala/scala/blob/78bcc844f1/src/reflect/scala/reflect/internal/Types.scala#L3709
Just a note: we'd prefer spacing here:

```ruby
Spree::Config[:track_inventory_levels] && !variant.on_demand
```
I'd recommend against overloading a boolean flag it this way.  Could you maybe add a string keyword argument `kind`, such that `kind=None` would correspond to `ordered=True`, `kind="unordered"` would correspond to `ordered=False`, while `kind="unique"` would correspond to `ordered=None`?  I think this will make the code a bit clearer.
> I disagree.  We should be able to organize the docstrings however seems best.  I also like to use "Example" when there is only one example.


I don't disagree here. Organize as you like, but use (like a library)
some pretty ubiquitous and meaningful headings for organization.

>
> Having a whitelist would be a major headache.  Any time someone edits a docstring header, they have to remember to update the

No. The whitelist would contain only the headings that we approve. And
if a heading is meaningful, it would likely (through peer review which
handles all things sympy) be approved, added, and no longer need to be
maintained. I would say it's a lot less of a hastle than making
additions to AUTHORS. (Note: I'm not trying to make this a sticking
point, but am just raising the point that I think a well thought of
list of acceptable headings leads to a more predictable doc reading
experience. It's just another application of "preferrably one obvious
way" of doing things.)
Oh. I guess they should be fixed too.  Sorry I didn't notice that in the other pull request.
->Examples
I vote for `f_1` at the end.
There is a conflict of interests here: we don't want to enable xy-pic by default, because some users might not have it installed, and thus preview might not work for no good reason. OTOH users who have it installed might find it annoying to always pass `package=['xy-pic']` whenever they want to preview a diagram. Do you see any way around this (e.g. is it possible to at least make an educated guess about whether or not certain packages are installed?)
DateInputFocusOut içinde dateInMilisecs gibi bir değer de döndürüyorum. Hazır olarak temin edilmesi işlem sürecini hızlandırabilir düşüncesindeyim. Herhangi bir başka component'in burayı dinledikten sonra bir de getDateTıme metodu ile değer istemesi gerekecek. Bunun yerine süreci daha otomatize hale getirmiş oluyorum. Bunda bir sakınca varsai çıkarabilirim, ama gayet mantıklı durduğunu düşünüyorum.
```
@param string|boolean $operator the operator used to concatenate where and having conditions. Defaults to 'AND'.

For backwards compatibility a boolean value can be passed. 'false' corresponds to 'OR', 'true' corresponds to 'false'
```
missing original word.

> (since 2.1)
No, but it's also a 2 second fix. I don't mind making the fix later if
necessary. I was just doing a 'review ALL the csbot pull requests' thing.

--
Regards,
Harry Mills
http://haeg.in
07708228571



On 4 December 2012 20:07, Andrei <notifications@github.com> wrote:

> In csbot.example.cfg:
>
> > @@ -48,3 +48,7 @@
> >  # This configuration is for the Example plugin
> >  [example]
> >  foo = bar
> > +
> > +[topic]
> > +separator = |
>
> Oh, cmon Haegin, this is not a big deal :D
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/csyork/csbot/pull/43/files#r2311797>.
>
>
let's use better variable names.. throughout.
Lists.newArrayList()
Thanks for the tip. I'll fix it tomorrow.
You will have to change Alexey parse function a bit for that.
Sneaky :-)
I've refactored this now so it is handled by a separate helper that is unit tested.
I think this block needs to either precede its description or not be nested in the final item of the list.
Cette ligne se répète dans tout les fichiers html. Il serait plus simple de la mettre dans *base.html*.
I'd prefer not to have another command line option where certain code could fail to work as expected if you don't set the flag. Can you explain what you're using the piped node data for and we can see if there's a way to make it work without the piped node data?
db_tools : currently not used within phpBB3, but may be utilized later, as written in docblock. Has this status changed?
Who's arguing?  I'm just stating the reason for doing it.  :-P  If it's not a valid reason then that's fine and I'll do whichever is preferred.
See comment on the other get function.
Use the existing language entry for this.
https://github.com/phpbb/phpbb3/blob/develop/phpBB/language/en/ucp.php#L177
You've added this exact code three times, is there anything that can be done about that?
I added it to a property because later when we get phrase search working for postgres fulltext search too, we can provide user an option to either use it or not use it. Property will make things more easy for this implementation. Does it necessary be made into a function returning a bool?
shouldn't this be called like, `_reconnecting` ?
I still think this log message needs to improve
this should be under !win32 sources -- its not portable to win32.
this view should be here only if it is restricted to cpanel users (in_staff_group). Else, move it to security.views.
That's mostly true. However, the `rollupFsPath` variable contains a path on the filesystem, and so should probably be using libpath.
26 sal az khoda omr gereftam hanooz nafahmidam chera age in 2 khato yeki konim khata mide !
Очень похоже что проверка $_POST['time_limit']>'0000-00-00' неправильная
на локалке нет инно, верни
I have no idea how i've done it D: I'll fix it
I believe this app.js file is unecessary, you can just use app/app.js
Don't call this Viewport, a better name would be AppCarousel
示例这里，可以给个链接么
On Thu, Aug 2, 2012 at 6:24 PM, Łukasz Langa
<reply@reply.github.com>
wrote:
>> +    if not settings.ZABBIX_URL:
>> +        return
>> +    zabbix = Zabbix(settings.ZABBIX_URL, settings.ZABBIX_USER,
>> +                    settings.ZABBIX_PASSWORD)
>> +    zabbix.set_host_templates(hostname, ip, templates,
>> +            group_name=settings.ZABBIX_DEFAULT_GROUP)
>> +
>> +
>> +def update_zabbix_templates(device, templates=None, remote=False):
>> +    hostname = device.name
>> +    role = device.venture_role
>> +    try:
>> +        ip = device.ipaddress_set.get(hostname=hostname)
>> +    except IPAddress.DoesNotExist:
>> +        return
>> +    if templates is None:
>
> What for? Am I missing something here?

With your code I would need a separate clear_zabbix_templates() function.


-- 
Radomir Dopieralski, http://sheep.art.pl
Common understanding by both Mark and me is that it is a step in wrong direction: making lexer/parser know more about promise types makes it less flexible (e.g. makes it impossible to add dynamically-loadable promise types).

The better approach to take here is to add/improve validation in mod_*.c layer, so particular promise types would be ultimately responsible for validation, not the centralized parser (it would require passing enough information to validators in promise type implementations).
All prototypes should be initialized to the same values as denoted by xml. They will need some internal mechanism to alter these values on a concrete instance-by-instance basis.
Be my guest - if you can get it working I'd love to include it in here.
I tried with  ``%<pun_include "[^\)]">%i``, it doesn't work anymore... But I'm sure there is a better regex to use, as I simply want to catch everything between the " ".
shouldn't the var statement be the first statement inside this block (so before the defintion of the `secondToggle()` function..?
Ok, thanks for the link - but I think you misunderstood the intention of my question, which is my fault for not being clear. 

In your patch, you removed the [line](https://github.com/jquery/jquery/pull/758/files#L0L174) where `type` was being set to "GET" and replaced it with an [initializer and no assignment](https://github.com/jquery/jquery/pull/758/files#L0R168). If you're taking advantage of the default behaviour of `jQuery.ajax()` that checks for `undefined` type, then **please** indicate that in a comment - "Default to a GET request" does not communicate the intention.
This actually fixes a heretofore unreported bug: http://jsfiddle.net/YHJ3D/ .
Open for discussion.  Maybe for now we have ```convex/lmf.hpp``` and ```task/lmf_task.hpp```.  I'm fine with alternatives, but my experience working on other projects is that having multiple files name ```xxx.yyy``` only leads to confusion.  Any differentiation we can add here is good.
As a novice, I wondered.
We preprocess/encode the original training table and get the encoded table, which is just a temporary table. It is not result table.
Is it possible that Eighen Sparse Vector may have 0-size vector?
It would be better if "href" was the first atrribute as in the <link> above.
Is throwing away this error for the majority of users a good idea?
Looks like your editor is using spaces instead of tabs. Would you mind fixing that and then I'll pull this in.
Since this is using a `.tar.xz` download, it needs a dependency on xz:

`depends_on 'xz' => :build`
Is there an extra newline there? Doesn't pep8 complain?
How does one atomically remove the messages from the list then though?
其实我是觉得make既不适合新手也不适合高手，只适合介于二者之间的要摸索成为高手的人，摸索过一遍明白这里面的道理了就没必要再这样整了。
(sorry about the delay; i have a busy day job so I can't respond daily)

I understand concurrency in Java very well. It looked like you were locking before every read and write from fListeners, so it didn't matter whether fListeners was thread-safe.
will `media.type` always be defined? if not, we should check for that as well.
Could we eliminate the 'doc' argument and just use 'element.ownerDocument'?
We can remove the array init here if we use the filter below.
The delta here _is_ removing console.dir. Is Github acting up again?
That's a brilliant observation! That's much better. I also created a function to generate the MenuItem id to make the code easier to maintain.

My code to allow multiple menu items with the same command in the same menu, made the id too complicated to do that simple lookup, but I forgot to revisit that when I backed  out that code.
For repositories in github, please use `:type github` and `:pkgname "USER/REPO"`.  You can also skip `:website` part.
Also, there are already checks for that in the CalculationValueUnitSpec, as
well as 1 in the CalculationUnitSpec, 1 in the ValueServiceSpec, and
ExpressionServiceSpec (marked as TODO), but true that this file does
manipulate partial values, and the ModeController does not.


On Mon, Dec 3, 2012 at 1:50 PM, François Terrier
<notifications@github.com>wrote:

> In test/integration/org/chai/kevin/value/RefreshValueServiceSpec.groovy:
>
> > +	def "test refresh mode refreshes all fields"() {
> > +		when:
> > +		def refreshed = new Date()
> > +		setupLocationTree()
> > +		def period = newPeriod()
> > +		def mode = newMode("1", CODE(2))
> > +		def partialValue = newModePartialValue(mode, period, Location.findByCode(BURERA), DataLocationType.findByCode(DISTRICT_HOSPITAL_GROUP), 0, v("1"))
> > +		
> > +		then:
> > +		ModePartialValue.count() == 1
> > +		
> > +		when:
> > +		refreshValueService.refreshCalculation(mode, new TestProgress());
> > +
> > +		then:
> > +		ModePartialValue.count() == 8
>
> Ha, ok. I just thought here would be maybe an easier place to put them
> since the partial values are already there (all that needs to be done is to
> actually check the content of those 8 values). Also the mode controller
> does not manipulate any partial values.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/fterrier/kevin/pull/267/files#r2291527>.
>
>
I don't think I understand this clearly enough to write it... if you write that part I can add it to the help (or you can just merge this pull request and add it yourself on master).
I was thinking that one way to go would be to simply call `annotation()`, but I think it would require some modification. Is there a reason why `annotation()` looks specifically for x, xmin, and xmax, but not other values, like xintercept?
Default 1 seems wrong as it wont even work IIRC plus the recommended value are multiples of 10 (times  max_cluster_size).

We are going for default of 80 in AS7. (jboss-as#3361)
What does this class do ? Why do we need it ? Why is it important for the clients of our APIs to know it exists ?
Ubuntu is not the same as Debian.
First. Like I had done for `LanguageField` for relation database.
Spacking around key, twice.
Yes, absolutely. Bad habit from writing CSS that wasn't destined to get minned.
Yeah that looks better to me.  Just see if it works.
If you add the JS and CSS to `localResources` you won't need this function
Nm. I get it. You need it open to check the rate limit.
Same as below, let's use logging.getLogger(__name__)
This should be local to the method processListingStreams
but that might be unreadable terrible craftsman code
This should be in "qoutes":

    #KATELLO_JOB_PARAMS="-m -p katello"

Otherwise you get

    -p: command not found

when uncommenting
This isn't actually used and shouldn't have been there.  Deleting :)
Yeah, I was just aiming to be explicit. I can just change it to `!charset` if you'd prefer.
Six of one… Keeping it as-is.
the visibility is missing, and you need to add an empty line between the attributes and the constructor
This does not test anything: this test will also pass with the current code as PHP does not complain about extra arguments :)
Yep, that'll go as well
I wish we don't have to digg too much inside the inner structure of the homescreen and use some properties provided by some objects.
Instead of keeping everything in a flat anonymous function, you can reorganize the code into module pattern or constructor function.

(this issue is non-blocking, can be done in a follow up)
(not your bad actually!)
Uh, okay. I'll just issue a PR against this one :)
I'm not sure which tests were failing but I didn't test this code particularly either. It's just what came to mind.
I would feel better, if this would be a std::string.
Shouldn't this happen first?  I remember in that in 5.2.x `isset($var['key']) on a string is always true.  I don't see why we need to accept mixed types here.
Probably there are no tests for non-existent files that's why there were no fails.
I didn't change the parser like I later did to ```length()``` (which is just working as a proxy for ```enumerationValues()```) because ```column()``` was only called [internally](https://github.com/cakephp/cakephp/pull/1011/files#L1R317).

Later I added this tests and managed them by simply [calling](https://github.com/cakephp/cakephp/pull/1011/files#L1R658) ```trim()```.

I realize now, this can be called externally, so I should test them too :)
Can't you use thru controller?
probably better rewritten as:

```python

    res = super(PostgresqlDatabase, self).connect()
    if self._schema is not None:
        self.execute('SET search_path TO %s', (self._schema,))
    return res
```
Wrm de check? Iets deletion dat niet bestaat is geen ramp.
Add NOTIFY signal.
Then I need to force `fsep` to be `/`, don't I. Because I don't want to have `\` in a URL when executing on a windows box.

What would be the advantage over `str_c` or `paste` (the help mentions speed but it's probably irrelevant here given the relative speed of constructing the url and actually downloading the file.
Neither me, but it seems other emus do the same. I have no clue, even after looking into docs for hooours.
Actually, if you try this:

```python
if not os.path.isabs(new):
    # If we receive a non-absolute path, make it absolute.
    abs_new = os.path.abspath(new)
    self.notebook_dir = abs_new
    return
self.log.info('Print this')
```

It logs. 
It behave as I assumed (but I admit I did not test that!): setting `self.notebook_dir` triggers a call to this function.
Since a call to this function took place, I used return to prevent a second run.
More to the point, code calling `reduce` is trickier to understand intuitively - that's why it was sequestered into a module for Python 3. I think this function is fine, although I'd describe the result as 'flattened' rather than 'concatenated'.
I prefer putting all this into an actual `main()` function and calling it from here.  It makes it possible to reuse this code from some other tool later on, rather than having a bunch of 'naked' top-level global code.
if you add Override annotation, it will break compile, alerting you that you are overloading as opposed to overriding the method.

can you not pass CloudStackEC2RegisterImageOptions to the old method sig?
please use style like Tag, which you can find/replace to remake this class in a consistent way with javadocs, equals, hashCode done correctly.
https://github.com/jclouds/jclouds/blob/824ea1fd0a25fdd825abd89927d875f358d4a69f/apis/ec2/src/main/java/org/jclouds/ec2/domain/Tag.java
PEP8 prefers load_devices as a function name.
We should be able to use SETNX to kill the race condition http://redis.io/commands/setnx ?
ahh yeah, that's way better.  I should have remembered that.  Added to the base class so I get it everywhere.
Fixed in the next push
Why a KeyError? (perfectly aware that I wrote this code originally but my head hard-drive ran out of space)
Why not a specific exception like "OOIDNotDefinedError" or something. 

At least a comment explaining why we'd expect a KeyError here.
`mongo`
In PHP (kind of at least) :)

``$this->pipes || defined('PHP_WINDOWS_VERSION_BUILD') and $this->fileHandlers``
no need to add the ``\`` and Symfony never use it for functions in the global namespace as it only uses functions from the global namespace anyway
> That's also why we have what we have right now. If you want to use translation, you need the translation component.

This doesn't change. The Default translator does not support message catalogs or any of this. It supports a simple subset of the Translation component that lets the Validator work without that component, unless you need actual translations or more complicated plural messages (which we don't have in the Validator core).
You should set the third argument to `true` in order to keep user defined value.
Can you remove this and submit it in another PR? Thanks.
Are diffs idempotent, or does this need to be done atomically with the forwardingTree puts?
Shorter lines, please
What if there is a dash in the name of the volume group?
Good question!  That's a question for @lgritz - Larry, can we change the ImageBuf::reres() API to take a spec rather than an ImageBuf?
Probably better as a lazy import at the time os use. The list can only grow.
It's the only way I could find of configuring raw_id_fields based on settings. If there's a better way I'll happily change it.
Did you change the path in the request, too?
Mark, Duncan,
 Does this look like better?
https://github.com/dreamhost/akanda/blob/user-api-with-quantum-extensions/akanda/userapi/addressbook.py

Yes, not sure how deep we want to go.
Is there something we should return instead?
ewww... backwards comparators!  Putting the constant value first when using binary operators is way unstandard and hard to read.

you should save the match.toLowerCase() and then compare them like this:

    ...
    match == 'all' ||
    match == ''everybody' ||
    match == 'everyone';
these should all be variabels anyway, for now they are hard coded. it migh tnot have been completley necesary to localize them
I don't really understand why we need a persistence provider adaptor that basically just does nothing. Personally I would prefer to see an approach where JPA providers that do not need any custom setup can simply use the spec required JPA API to bootstrap.
May as well add a space after the `//` whilst we here.
You don't need this change so please revert it.
Use either `hidden` attribute or class name to toggle the visibility, avoid change the style directly.
Please, move this comment on top of L38 remove the dead code (`else {` and `}`)

Why would you want to do this?
can’t you just update the `location.hash` instead?
put `{` in the previous line.
that was also my first thought, but I saw that Homescreen is not using any classnames. Anyway, I'll fix it.
Can you add the default values for those in build/settings.js too?
please reference the bugzilla bug number
I assume we would want to avoid any transition or delay, the homescreen has crashed while it is running.

I don't pretend to understand all the transition logic but most of it seems centered around the fact we want to display
something over the home-screen (which in this case has crashed).
nit: i don't think this line is needed.
nit: favList -> favoritesList
are you waiting for the adds to finish? feels fragile
This method is **not** defined in the dialer, so we need to remove it.h
Shouldn’t this `custom_dialog.css` file be replaced by `shared/style/custom_prompt.css`?
Same as above. Should probably make this parsing piece its own function.
Maybe skip this line altogether if it's null/empty?
I am always paranoid when seeing acquire and release not close together.   For instance, not sure who/when the lock is owned.
I'd say it was material for the pull-req cover note/comment

mad props to @nuclearsandwich for getting things in motion
When you have variables that don't need to be exposed outside this scope, I prefer to use locals rather than ivars (which could conflict with other variables used in the including scope).  You can use a local like so:

``` ruby
some_var = nil
before do
  some_var = some_value
end

after do
  do_something_with some_var
end
```

Essentially, declare a local in the parent scope of both blocks so they can both close over it and use it.
Observing the 'id' property is exactly what this mixin does ... but if `DidPersistRecordObservable` is clearer for you, I´ll change it
I need better is logged in than this
Not really need as you have same `return` just below.
Your `render_data` is generally referred to as `context` in the django template world.
oh man, I hate @simplyianm  for that..

will change it after I get a go for this.
I never used it and never had problems.