labofsahil/github-event-dataset-2012 huggingface dataset:
These changes do not include net02's changes.  When I tried to merge them, they seemed incompatible.

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.
rowend/12-admin-sale

complete admin sale folder translation
Issue when rendering if a option has a '-' in  a Jinja2 template.

i have problem when rendering the option 'target-file'.
So the recipe is aware of 'input' ('template-file') and 'output' ('target-file') option as well.
Dynamo improvements

1) Dial down dynamo throughput to free usage tier
2) Enable integration test for dynamo Java client
3) Upgrade AWS SDK to 1.3.26
merged part of PLFM-1362
Fix for #100, runserver_plus doesn't serve static files from apps

https://github.com/django-extensions/django-extensions/issues/100
rowend/31-catalog-affiliate

Transalate catalog affiliate folder
Atom feed fix

Atom.xml and head are done. Needs a merge
Support for region argument

Hi,
I have added the support for region input argument.
Please include it or let me know if you need any modification.
Thanks,
Niket
Install Script Sanity check :P

IPB DB API utilized fully and not attempting to add/drop a field when it may potentially error, and already has on an individuals board.... double installation is possible in hooks and all that.
Long lines support added

'\' at the end of the line can be used, not only in preprocessor
definitions..
Allow postfix const/immutable/shared/inout for delegate type

The syntax change is necessary, because it has been allowed from 2.061.
Added working window re-sizing
get sample programs
Cleaning up removing views and passport

We don't need views or passport on the auth sevice
Use filebrowser MEDIA_ROOT and MEDIA_URL settings.
Watch out for variable name clash; you've got $user used in two places here, and with assign-by-reference, you can sometimes get hard to debug errors down the road.
With array operations just say U_SPUG = 0.0_DP rather than say the limits.
In a module to handle events, kinda google calendar, we can set the all day event, so I use the function get_day_start and get_day_end to set the start day event and the end day event, is a useful util lib for that purpose
I had that in the beginning but this gets huge. There are tens of actions to be supported here. I'd rather leave it like that.
Actually i think the other commits in the first merge do already. If there are any (bad) inconsistencies at all. It might very well be that your versions are much less fucked up than mine ;)
First i wanted to answer: "because i didn't want to add more bugs" - but since i wasn't even able to manage that, i can only ask if you want me to revert it or fix the typo? ;)
You could make `b` an int, and assert that the resource destructor runs exactly the number of times you expect.
As long as the exit window is available. It's fine to me.
i'm pretty sure i remember EHLO from back in the 90s. it's too new for some servers?
Revert intended?
Unfortunately that patch breaks a community style guideline regarding the 80 column limit. Quick patch: https://github.com/tim-smart/node/commit/e4af3e93cad897753ba4fad14733eee7e03dee4c

(Will post this on the pull request also)
I just found out that the `"wb"` mode would be exactly equivalent to `File::Constants::CREAT | File::Constants::TRUNC | File::Constants::WRONLY, binmode: true` anyway (I somehow missed the TRUNC, thus only overwriting beginnings of files on subsequent open/write cycles), and this is an acceptable workaround in my case.
Baby tears are the fuel that powers me through the day.
Weird, im pretty sure it was ok even in the compilation but today I'll start over again the script to manage the proper npc movement that accidentally discovered yesterday
Still generating as much noise as possible for mendibot to catch via ATOM
In my tests it seemed to be. But then again, I created a new one for each window open so it may have just been the fact that I was calling it 15 times and not that it is particularly expensive.
Ik denk dat de site binnenkort toch overal `jquery` gaat gebruiken.  Maar het zelf schrijven is inderdaad tof.
auth=true line not needed
But this is not hospital. Every bigger project work like this. People makes mistakes and it's normal to point them out and help them understand what they did bad. Every bigger project has better coders and worse coders and every of them makes mistakes, but I 've never seen before in any project, that somebody was pointing again and again who made the mistakes.. Just correct it and it's fine, the one who did mistakes will know it.
What about replacing this conditional with `@violations.merge Array(body).map(&:line)`?
Changing history during review can be an issue. Perhaps the correct approach is to keep your original branch, and a second one with name "<origname>-rebased" that you also keep on github and link to from the PR discussion.
Se te colaron ficheros me parece ... otra vez un commit -a ? x.3
You can't make the assumption that the `UnitTest` class above has already been set up and run:

    $ bin/testify test.test_case_test UnitTestTestYoDawg
    ERROR    fail: test.test_case_test UnitTestTestYoDawg.test_unit_test_status
    Traceback (most recent call last):
      File "./test/test_case_test.py", line 299, in test_unit_test_status
        assert UnitTest.status == [True] * 6, UnitTest.status
    AssertionError: [False, False, False, False, False, False]
    
    F
    FAILED.  1 test / 1 case: 0 passed, 1 failed.  (Total test time 0.00s)

It also doesn't always work when bucketing tests:

    $ bin/testify test --bucket=1 --bucket-count=2
    ............ERROR    fail: test.test_case_test UnitTestTestYoDawg.test_unit_test_status
    Traceback (most recent call last):
      File "./test/test_case_test.py", line 299, in test_unit_test_status
        assert UnitTest.status == [True] * 6, UnitTest.status
    AssertionError: [False, False, False, False, False, False]
    
    F...........
    FAILED.  24 tests / 15 cases: 23 passed, 1 failed.  (Total test time 0.00s)

Or in client-server mode:

    $ bin/testify test --serve 7357 & bin/testify test --connect localhost:7357 >/dev/null & bin/testify test --connect localhost:7357 >/dev/null
    [4] 31961
    [5] 31962
    ERROR    fail: test.test_case_test UnitTestTestYoDawg.test_unit_test_status
    Traceback (most recent call last):
      File "./test/test_case_test.py", line 299, in test_unit_test_status
        assert UnitTest.status == [True] * 6, UnitTest.status
    AssertionError: [False, False, False, False, False, False]

    ERROR    fail: test.test_case_test UnitTestTestYoDawg.test_unit_test_status
    Traceback (most recent call last):
      File "./test/test_case_test.py", line 299, in test_unit_test_status
        assert UnitTest.status == [True] * 6, UnitTest.status
    AssertionError: [False, False, False, False, False, False]

    F
I already have audio and faster graphics can't really be done unless I write a WebGL shader, or something.

The performance improvement wouldn't be that large.
That's probably more pedantic then necessary. C89 clearly accepts var declarations inside any block declaration start. Guess I'll need to fix my script... :(
Ah.  I found this issue when reordering function declarations at top level, where the missing delimiter meant that the first function didn't get a new line after it.  In general when the delimiter is a newline, inserting it always is the right thing to do.  But I can see how a different behavior might be warranted for commas.  Not sure what the right thing to do is, beside adding another 'always delimit' optional parameter (but that seems inelegant).
Placeholder to avoid unused variable messages of some compilers. Another solution would be to remove the names from the function declaration.
Out of curiosity, would the following issue be solved:

* http://na-mic.org/Mantis/view.php?id=1481
* http://na-mic.org/Mantis/view.php?id=1482
No freezes for now... thank u
CMSG_GUILD_UPDATE_PARTY_STATE                    = 0x02219, // Checked on: 4.0.6a 13623
@alloy thanks. I fixed the homepage url

i'll sure to include a spec file on the project repo as well. to use the cocoa spec from the repo, i'll have to manually run:

pod repo https://github.com/aporat/three20lite.git

or there's some sort of auto spec files detection built into cocoa pods?
Or `write(out, "0x")`.
Небольшие изменения, связанные в основном с возобновлением огня при возвращении игрока.
Without timeout, if MozSettings fails, the keyboard would failed to 
show up.
And MozSettings is currently failing right now.

Since keyboard is a mission critical part for testing (as John points 
out on the mailing list), I think it worthy of a fallback check and not 
to depend on other part this much.
baf35d9
See comment on a previous patch, typo (illegal)
getColorClassesHTML() braucht ne timeline instanz.
man könnte die natürlich auch einfach static setzen...
much intrest - first (one string) commit. supposed problem with broken sessions.
:) 1300-1580 - heavy unstable period...
Thanks for that, I am made the change.

On Dec 20, 2011, at 4:58 PM, Charl Matthee wrote:

> This is not a backwards compatible fix. I suggest:
> 
> 144     def xor_str(str1, str2)
> 145       i = 0
> 146       newstr = ""
> 147       str2 = str2.bytes.to_a
> 148       str1.each_byte do |c1|
> 149         c2 = str2[i]
> 150         newstr = newstr << (c1 ^ c2)
> 151         i = i+1
> 152       end
> 153       newstr
> 154     end
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/npink/radiustar/commit/dd9d49213e55a755d4eedf00b6c4f62303fde615#commitcomment-811234
There is only 1 tap in that case. From memory there was many tap included in the problem. Or I'm mixing the different problem/fix?
Resets the value of data_len to 0 everytime the function is called. Was causing array overflow after 100 bytes where received.
Should be `c - 'a' + 10;`
r+ sweet then looks good
Ocelot also uses 1.0.14 so this last fix was necessary there - and so tiny as to not be a big deal.  I will endeavor to stay compatible with >= 1.0.14 for now.
Good, this is something I've been meaning to do for a while.
It was in the original tutorial to be instructive I assume. I took it out.
Welp, looks like setenv isn't part of standard c++. I'll remove it from my branch too, and try to find another way of doing things.
A mudança foi só essa, mais o "code style" do arkcore é horrível.
EN: The change was only that, but the "code style" of arkcore is horrible.
Should be PEAR_Exception, not Pear_Exception. Is there a relevant require_once() for it?
If not, I'd consider adding a File_DNS_Exception.
``volatile``, so it does not change during the cloning process?
Thanks for issuing these bugs by the way it's great to have somebody raise windows problems for me.

I'm surprised it even works for you :)
My first impression is that `connect2` is even more complicated than the current `$$`. But after I groked its flow, I saw that it's actually very simple and you just moved the "check the buffer" thing outside the hotpath.  I like this approach and would just appreciate some comments on `connect2` (as always, I don't know exactly which comments I'd like to see =]) and putting that big case on `bsState` inside the `where` clause.

The best thing about this commit is that it ditches `bsourceUnpull`. Now conduits are ready to rock ;-).
Almost forgot: I also removed the use of basename() since it was vulnerable to name clashes - e.g.:
* when multiple libraries have a "variables.less" file;
* and there is a single "main.less" stylesheet importing from all libraries.

In my use cases I detected no problems about that either and I was careful to ensure all bits and pieces received as filename the name actually used by the loading logic (for both browser and node).
When using Visual Studio, CMAKE_BUILD_TYPE isn't defined and can't be used.
A few possible ideas:
1. Have two separate variables for passing around the debug and release plugins.
2. Do a file(GLOB ...) in the plugins directory.
3. Don't support debug and only run the test in release mode.
oh no, it *is* voodoo.  I'll add a line comment.

The daemons use `fs.watchFile()` which is set to poll tightly when os support for detecting when a path's existence changes.  300 is very generous, but I wanted to allow enough time so that even in a slow machine, or one under load, there would be a very slim chance that the drivers state wouldn't change in time for the tests.
SO. EXCITED.
Can't we put this in the last stage of the market submission and then we don't break addons at all?
was it a bug, or just coding style?
Ok, i'll do it during the merge.
That kinda seems like cheating though. Like switching from piano to trumpet by using a synth  :)
Okay, the hosted servers don't have the latest code from the development branch which have a new API feature used by the import script.

See http://groups.google.com/group/openphoto/browse_thread/thread/31db09b87c2724de
It's only needed  to fix linker errors when building the tests.
 Clang does not seem to support gcc's __thread specifier, which seems to by used by gtest.
The -pthread flag cflags enables in libc some subs.
But I will check if the usage of __thread specifier in gtest could be suppressed.
yeah, I like it
see a440a3537153a23ad95215d443de72c3168629a5
Unfortunately HTTP/1.0 is still being used used. Squid proxy (quite popular proxy at a time) didn't support HTTP/1.1 up to version 3.1: http://wiki.squid-cache.org/Features/HTTP11 ([another source](http://serverfault.com/questions/58803/how-to-configure-squid-to-send-back-a-http-1-1-response-for-a-http-1-1-request)). It's your choice to rule out http/1.0 if you wish.
I hope eGit die merge everything fine. You did too much work :P
We could add a property, world-height, which allowed the server owner to change height, but based on sections.
Min height must be 8, as that leaves 127. Thus, default is 16. Or is this too much?
I thought you'd enjoy it.  :)

Sent from my iPhone

On Jan 27, 2012, at 8:05 PM, "toantran" <reply@reply.github.com> wrote:

> Haha, thanks Cole.  This is awesome!
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/SuperCole/wifefoos/commit/c26a6a038462bb3f58dcde5ed21daf0d08b3673c#commitcomment-910340
Locknes, that is already fixed on head.
reviewed and approved! great job
On second thought, (__obj->*functor)(m_iterator) is the origin of the false value return. Thus it knows why it broke the loop :-)
I know this commit is more pythonic, but we actually taught a class at a hackerspace to people who were new to python.  To almost everyone, the logic seemed backward.  Not that your implementation is wrong, in fact both methods will work (with newer versions of simplecv, 1.3 will be released shortly), although to run the feature detection you also have to be running the development version of SimpleCV.
there seem to be quite a few issues in the CL. I tried to review it quickly, so I'm sure that I didn't catch all of them. Maybe @vojtajina should do another pass?
Yup, that works too. I dont really care which as long as a fix gets merged
Will, do you have different line endings or something?
fucking php
This should be const.  I'll send you a patch ( #15).  The rest, APPROVED.
As it is written here, it will be 45 to 75 minutes.
Adding a comment on 2nd line
what do you mean by "social disciplines"? sounds very much like "gesellschaftliches Benehmen".
couldn't you just wrap the non-safe code in some `mutex`es or something even smarter (Active objects I read about)? 
btw, possibly ofLog gets some overhaul soon (due to other issues), so maybe now is the time to pitch in: https://github.com/openframeworks/openFrameworks/issues/858
No i haven't had any issues with thread safety, yet. Haven't had to deal with it, anyway.
Good catch, but technically this is a numeric comparison so it ought to be using `-ne 0`, not `!= "0"`.
Precisa sim.
Pois ele só pode pegar o resto da divisão por 11 caso o 'total' seja maior ou igual a 11.

poderia ter feito a mesma coisa utilizando o IF também, ficaria assim:

total = (total % 11) if total > 11
Wait. What the actual fuck.
should it be "with status: %s" now?
Thankfully it should be very easy to add concurrency using another gem called [Parallel](https://github.com/grosser/parallel)

```ruby
Backup::Dependency.load("parallel")

def perform!
  directories.each do |directory|
    Logger.message("#{ self.class } started syncing '#{ directory }'.")

    Parallel.each(hashes_for_directory(directory), :in_threads => 50) do |full_path, md5|
      relative_path = full_path.gsub %r{^#{directory}},
        directory.split('/').last
      remote_path   = "#{path}/#{relative_path}".gsub(/^\//, '')

      bucket_object.files.create(
        :key  => remote_path,
        :body => File.open(full_path)
      ) unless remote_hashes[remote_path] == md5
    end
  end
end
```

Basically replace `hashes_for_directory(directory).each` with `Parallel.each(hashes_for_directory(directory), :in_threads => 50)`. However, it seems you're first going to iterate through a single directory, and then transfer the files within that directory. If people have a lot of nested directories with just a few files in them (think of Paperclip/Carrierwave uploads) then it'll still be relatively slow since it will likely only spin up 2-3 threads at a time and then wait for the next directory.

This approach seems fine for single-threaded operations. The only thing that takes a while to process is the actual `bucket_object.files.create()`. So maybe it's an idea to, rather than perform the transfer operation immediately, to instead store the data in an array of Procs. I'm not sure if it'll work, but basically what we want is to "freeze" the `bucket_object.files.create()` operation so we can perform them massively parallel later.

Again im not sure if this will work, but you could try it:

```ruby
Backup::Dependency.load("parallel")

def perform!
  operations = Array.new

  directories.each do |directory|
    Logger.message("#{ self.class } started syncing '#{ directory }'.")

    hashes_for_directory(directory).each do |full_path, md5|
      relative_path = full_path.gsub %r{^#{directory}},
        directory.split('/').last
      remote_path   = "#{path}/#{relative_path}".gsub(/^\//, '')

      operations << Proc.new do
        bucket_object.files.create(
          :key  => remote_path,
          :body => File.open(full_path)
        ) unless remote_hashes[remote_path] == md5
      end
    end
  end

  Parallel.each(operations, :in_threads => 50) do |operation|
    operation.call # perfoms the pre-built transfer operations in 50 threads
  end
end
```

While Parallel is quite simple, it's very powerful. Let me know if you have any questions or need any help with this. :)
Without this change messages appear in log files as "<Info>: [object Object]"
@blackcat2 in retail its not generated, its made by players
I don't think so.  
Loop counts and global loop termination conditions are currently mutually exclusive options (I think).  
If you could specify a counter and a loop termination condition it would terminate if either were true - so the code as it stands would be correct.
Did I accidentally delete your comment @kangax?

You asked:

> Won't this prevent collapsing of whitespace in labels (which unlike in textareas and pre elements is insignificant)?

My intended reply was:

Unfortunately the following are rendered differently:

```<label>Label: <input></label>```

```<label>Label:<input></label>```

The space is in this case significant, which is to say that spaces can be collapsed but inner spaces cannot be trimmed.
nice
```
DELETE FROM `achievement_criteria_data` WHERE criteria_id IN (5220, 5215, 5218, 5221, 5216, 5219, 5213, 5018, 5217, 5214, 5233, 5235, 5232, 5234, 5238, 5236, 5229, 5237, 5230, 5231);
-- Classes
INSERT INTO `achievement_criteria_data` VALUES (5220, 21, 1, 0, ''); -- Realm First! Level 80 Warrior
INSERT INTO `achievement_criteria_data` VALUES (5215, 21, 2, 0, ''); -- Realm First! Level 80 Paladin
INSERT INTO `achievement_criteria_data` VALUES (5218, 21, 3, 0, ''); -- Realm First! Level 80 Hunter
INSERT INTO `achievement_criteria_data` VALUES (5221, 21, 4, 0, ''); -- Realm First! Level 80 Rogue
INSERT INTO `achievement_criteria_data` VALUES (5216, 21, 5, 0, ''); -- Realm First! Level 80 Priest
INSERT INTO `achievement_criteria_data` VALUES (5219, 21, 6, 0, ''); -- Realm First! Level 80 Death Knight
INSERT INTO `achievement_criteria_data` VALUES (5213, 21, 7, 0, ''); -- Realm First! Level 80 Shaman
INSERT INTO `achievement_criteria_data` VALUES (5018, 21, 8, 0, ''); -- Realm First! Level 80 Mage
INSERT INTO `achievement_criteria_data` VALUES (5217, 21, 9, 0, ''); -- Realm First! Level 80 Warlock
INSERT INTO `achievement_criteria_data` VALUES (5214, 21, 11, 0, ''); -- Realm First! Level 80 Druid
-- Races
INSERT INTO `achievement_criteria_data` VALUES (5233, 21, 0, 1, ''); -- Realm First! Level 80 Human
INSERT INTO `achievement_criteria_data` VALUES (5235, 21, 0, 2, ''); -- Realm First! Level 80 Orc
INSERT INTO `achievement_criteria_data` VALUES (5232, 21, 0, 3, ''); -- Realm First! Level 80 Dwarf
INSERT INTO `achievement_criteria_data` VALUES (5234, 21, 0, 4, ''); -- Realm First! Level 80 Night Elf
INSERT INTO `achievement_criteria_data` VALUES (5238, 21, 0, 5, ''); -- Realm First! Level 80 Undead
INSERT INTO `achievement_criteria_data` VALUES (5236, 21, 0, 6, ''); -- Realm First! Level 80 Tauren
INSERT INTO `achievement_criteria_data` VALUES (5229, 21, 0, 7, ''); -- Realm First! Level 80 Gnome
INSERT INTO `achievement_criteria_data` VALUES (5237, 21, 0, 8, ''); -- Realm First! Level 80 Troll
INSERT INTO `achievement_criteria_data` VALUES (5230, 21, 0, 10, ''); -- Realm First! Level 80 Blood Elf
INSERT INTO `achievement_criteria_data` VALUES (5231, 21, 0, 11, ''); -- Realm First! Level 80 Draenei
```
@ProjectEmerald: can you tell me which revision, whether your "unloadgrid" setting in config is set to 0 or 1, what's the OS on machine where it runs and what's the diff when 800+ players are logged in?
not the clearest of variable names.  maybe gatheredAttributes or something else.
Thanks, Nils!

Neither of the % projects build on windows, I suspect it is related to the
% character but I'm not sure.  This has nothing to do with your work.

I updated the jcom.dataspace project and (with a couple tweaks) it works
like a charm -- thanks for doing this, Nils!

best,
  Tim


On Thu, Jan 12, 2012 at 10:49 PM, Nils Peters <
reply@reply.github.com
> wrote:

> I just pushed this feature. Could you please compile jcom.stencil% and see
> if the macro works on windows?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/jamoma/JamomaFoundation/commit/6994a71ce1669898f6cc87de1358a7fb992e499a#commitcomment-863486
>
The second parameter of fread is the size and third parameter is nitems.  On success case it will return nitems.  Since one of the input is 1 so the above fix will work fine but I'm just curious if we should switch for correctness and check for 1 as the return value.  If the size is really the number of items then the above fix is fine. Just throwing something out there... no biggie!!! :-)
Нахуй это если метод compareGroups депрекейтед?
+1 same thing as with the routes ...
correct the spellings for method names to be consistent
Maybe inline.
I guess you could stay with https. At least for me there's a redirect anyway.
no! it's for China!
If/when it becomes possible to re-prioritise superclass renderers, this will change the priority on all occurrences of that renderer in all subclasses of that superclass. (So if I inherit JSONView and change the priority of json for that one view, all other views that inherit JSONView will have the renderer re-prioritised, too).

I'm not quite sure of the most elegant way to get round this at the moment!
Sure.  It seemed like this made a better-looking "link" than the `@link` command did.  If this is not a good way to link to the page, we should change it.
It might be the same as Gluth's Decimate spell.
Same here... does it make sense to return EXIT_SUCCESS?
Jag tycker också det verkar väldigt krångligt, jag kollade lite i PHP-manualen. Först testade jag bara session_destroy(); men det räckte inte, så jag kopierade bara exempelkoden de hade där. Den är som sagt väldigt provisorisk, tills jag kunde fråga dig om en bättre lösning. ;) Jag ville bara slippa stänga av webbläsaren varje gång jag ville logga ut.
Can we get these tests merged soon? I'd love to have them in master.
oh, i forgot, you probably have to change it on the solution or project as monodevelop has solution or project specific formatting settings (which they use the global settings as defaults for the first time solution is opened). right click on solution or project and do options.
for P1000 and P1000N change to CAMERA_ID_MAX = 2
Where is the delete your comment button, ah found it.
Please add POD for this Action too.
You can also use "&&" operators in your LINQ-Queries, you don't have to use .Where(...).Where(...)
IE n'arrive pas à accéder à indexOf pour lui mettre le langage par défaut. J'ai essayé de créer une fonction indexOf mais j'ai mal du m'y prendre.
It's better to hook this into wp_enqueue_scripts instead of running it at admin_init
Well in CSE 143 I got dinged a point for having more than 80 characters on a line! What do you mean by "It's also fine to wrap it to lines, right?"
On Jan 6, 2012, at 4:25 PM, Steve Loeppky wrote:

> What's the 80 character limit?  80 charceters is way too narrow for modern screens.  How about 120 or 160?  It's also fine to wrap it to lines, right?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/BigLep/ChuckanutBayFoodsInternal/commit/1fc37369a88561934954c8ce0098b30a1261a930#commitcomment-845378
don't need notes anymore, git should handle it.
looks nice, but I don't think it is consistent with the current _Validator_ and validate methods.
nome do método esta errado, depois vou trocar para alterar_matricula, pq ele pode tanto add quanto excluir turmas
this is a fix for issue #32: https://github.com/socialcast/devise_oauth2_providable/issues/32
Removing the audio ifdef. The one you claimed fixed it.
On Feb 23, 2012 11:34 AM, "LorD ClockaN" <
reply@reply.github.com>
wrote:

> so you are telling me that you have working mic with new drivers??
> you remember what commit fixed it?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/TwistedPlayground/android_frameworks_base/commit/830100747ee6630fbedfda77c83e132d9fd90c81#commitcomment-1003173
>
Yes, its a governor so no kernel dependencies. :)
Make sure that you've included the attribute-value pair `data-rel="dialog"` on your link to the sub-page dialog. Because of the way jQM handles linking to dialogs, the `data-rel` attribute is still necessary on the link even though the dialog div's `data-role` attribute also says that it's to be treated as a dialog.
Я не понял профита. :( Покажи на этом коде, как ты видишь, пожалуйста.
In the mean time I've made some changes to the public_url so it will fail if the file doesn't exist.
I'm leaning towards `options.return`. This would add an additional check for isFunction, but allows Y.Color to be more versatile than it already is.

After that I would go with `options.toString`
Shouldn't it be "pmUtils.designDirty === false"?
So until we're using petsc for GPU solves, I think we go ahead and keep disjoint names.  I can fix up the cusp_solver code so that preconditioner: None correctly maps to no preconditioning.
twsited -> twisted
oh, now I remember what that was which has been nagging me
AFAIK Akka doesn't do any load balancing of nodes yet. Do you mean that it's primarily used for load-balanced routers?
not production quality, but good enough for build, I guess
Beer? :D
ok, I can cleanup that in my other router work, but leave this as small as possible to simplify backporting
NonFatal?
or is there a reason for `e: Exception`?
perhaps call super.unhandled first?
In addition: Removal did not break anything, hence it was not used. And there is always the val default available, which is more expressive than apply() returning a cached default, so the removal increases intuition.
I tried to say that the order *will* be:

        testConductor.removeNode(third)
        testConductor.shutdown(third, 0)
ok, took care of that edge case
I would love to learn rebase, I still do not fully grasp it and I think it would be very useful to learn so I welcome your help and guidance.
Simply raising NotImplementedError() should be fine by itself.
I know this is because of the query=[] in the method params, but might it be cleaner and raise less questions if the method signature had query=None and it was explicitly checked?

Am happy enough if you think it isn't confusing though and the rest looks good.
from ckan... import * is evil

please never do this :)
These should not be hardcoded, different extensions could define their own facets so these will limit them to this list.
this doesn't make sense, does it?  if there's an id, how can it be missing?
Hmm.. why is pts wrong here after a seek? 

it's possible that just checking m_started variable would solve the issue. A seek should cause m_started to go to false, when true again pts parameter aught to be valid again. But i still wonder why pts is wrong there to start with.
Oh, nice. Sorry, I didn't get this far before talking. :-)
Looks like we could use path.sep here: http://nodejs.org/api/path.html#path_path_sep
<< endl;
yes, that’s better
sur la table
I've added support for Twig_{Function,Filter}_Method in 98e9e98, however it's untested.
Needs to line up with the `String::New(...)` statement.
I'd rather this be called `rotate` and the other one be called `rotateObject…`.
So I finally had a chance to look into this, and I think it's an artifact of how Scala creates the closure objects, which leads to a reference to the outer class existing in the closure. Unfortunately, it's not easy to fix this, but if you're curious, there is a workaround:

1) Move DataPoint to be a top-level class in the file, instead of a nested class

2) Right before the foreach, add local variables that reference the variables from the outer scopes:

        val weightDelta_ = weightDelta
        val data_ = data
        val weights_ = weights

Then use these variables in your closure instead of the outer ones.

This helps because the body of each for() loop is actually a closure, and doing this prevents us from having to reference the outer for() loop. Similarly, with DataPoint being an inner class inside the method, each DataPoint instance could actually hold a reference to the whole Suite.

However, I would actually not include the test, because I realize that the value being sent across is an artifact of local mode, and is not going to work when you run on a cluster. This is because value is transient in Accumulable. Looking back at why we did this, reason was actually that in some cases the value is large (e.g. a hash table) and is not used on the worker nodes, so this will waste a lot of network bandwidth. So I'd rather make it an error to access value on the cluster, and recommend that for applications that do want to use it, they assign it to a local variable. For example:

    val accum = sc.accumulator(...)
    for (i <- 1 to ITERATIONS) {
      val oldValue = accum.value
      data.foreach( /* do stuff with oldValue */ )
    }

Does this sound okay? If so, I'd like to merge this pull request, because there's a lot of useful stuff here, and we can figure out the value thing later. (Maybe we can add another class for when you do want to see the old value).
"which is real due to Hermitian symmetry.
Isso aqui tá errado
оставил пока как есть.
явное преобразование к QVariant можно убрать
I see. I guess this was a misunderstanding on how the Rails.queue was meant to be used. Are you suggesting something like:

```ruby
def initializer(method_name, *args)
  ...
  @queue = Rails.queue
end
```
`split_paragraphs` is not private? I think we don't need to test private methods.
You're absolutely right. Do you think I should add it to the same test method or make a separate one?
I think so. I did them separately to ease the understanding of what's going on, but it would be easy to do in one step.
Is it OK for transaction levels to be `-1`?
`remove` accepts many table names, but it doesn't use any special option to change their behavior. `remove_index` accepts only options.

I prefer the `:polymorphic` check to the `column_exists?` check.

That said I would not change the API.
Yeah I saw the commit, no worries, thanks!


On Sun, Nov 11, 2012 at 12:14 PM, Nikita Afanasenko <
notifications@github.com> wrote:

> In activesupport/lib/active_support/core_ext/hash/conversions.rb:
>
> >              if value['type'] == 'array'
> >                _, entries = Array.wrap(value.detect { |k,v| not v.is_a?(String) })
> >                if entries.nil? || (c = value['__content__'] && c.blank?)
> >                  []
> >                else
> > -                case entries.class.to_s   # something weird with classes not matching here.  maybe singleton methods breaking is_a?
> > -                when 'Array'
> > +                case entries  # something weird with classes not matching here.  maybe singleton methods breaking is_a?
>
> Actually this comment was added in @9e44614<https://github.com/rails/rails/commit/9e4461438f8ce584b635aca35579c36537a340ca>,
> I do not think we need it anymore.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rails/rails/pull/8173/files#r2092644>.
>
>



-- 
At.
Carlos Antonio
When the `Parser` is not available?
This slipped into the commit, let me know if I need to (boringly) restore the spurious whitespace :)
I could interate over an array containing the three symbols though
Well let's hope it's not !wasDelayed && !nowDelayed, since this would overturn most of what we know about logic.
There is a missing (2) on the diagram?
Yeah, I was thinking the same thing as I wrote it.  Probably not.
Is this a regression/ Or perhaps you should define x as real and still include these tests (if not being real is the reason they were deleted). So try preserve the original intent of the test or else something will become uncovered (perhaps).
`x = Symbol('2')` means that you create symbol (object of class `Symbol`) with the name '2' and representation as '2'. And assign it to the pythonic variable `x`.
But `S('2')` create `Integer(2)`.

Compare:

```python
>>> Symbol('2').name  # it is confusing 
2
>>> Symbol('x').name  
x
>>> Symbol('2') + 3     # it is confusing 
2 + 3
>>> S('2') + 3
5
>>> Symbol('x') + 3
x + 3
>>> S('2 + 3')
5
```
studys->studies
There is a small problem with this change. The model attribute `password` is currently used for both password form input and to store the password hash/salt in the database. If we change it to `password_hash` then I think we need to add a public property-based attribute to store the password input from the form. While that would make sense I am concerned it might affect the tutorial elsewhere.
better to check isset($_SERVER['SERVER_PROTOCOL'])
Done. 
6ca3fecb218355550c54a26a8e69cfc9e98944b3
omit original text?

> (``$this->ModelName->id = $id``)
@return L{Histogram}
It's not calling a function, and the string is fetch in the compile pass, so it's better, faster, stronger
Ok, I chose to delete them as the options are used as options for the finder method.
Shouldn't this be at the top?
I'd rather minimize information loss... if setup/teardown should match for each test in a module, at worst you'd have an upper bound on that time (duration of the shortest test of the module) and relative total performance of each test (and would benefit tremendously from having setup/teardown aggregated over each module for estimating the per-test exclusive time as `[inclusive time] - [module aggregate setup/teardown] / [module test count]`). But if setup/teardown don't match, there's no way to recover the total cost of a test from only it's exclusive time and the total module duration.

Granted, that latter case is almost certainly bad test design that should be refactored, so I'm not sure what to rely upon other than my personal use case. @Krinkle, can you think of anyone else who might provide valuable feedback?
нужно брать эту роуты. вместо них нужно оставить:

    resources :events
    namespace :user do
      resources :events # это вложенный  ресурс, контроллера для этого случая нет, поэтому его нужно создать тебе самому
    end
What does this line do? ;) need description
Well I made this more relaxed than to_param.

to_param will only accept q5v3 for example.

The above regex will accept q5v3, q.5v3, q. 5v3, q. 5 v 3, q. 5 v3, q. 5v 3 and without a version: q5, q.5, q. 5 and other variations with/without whitespaces.
the `syncedTo` field in the output document of <method>
nit, after Craig's HTML+CSS 101 - can we make this class simply 'template' ?
Found the problem.
This function is never called.
This is a failure return, yet you 1) did not do anything to fail and 2) did not populate $error. If form preparation is optional and there are avatar drivers that don't need it, return true. Otherwise delete this function per above.
both are okay, we also start not adding the "? true : false" anymore
However, since you do appear to be passing this command through a shell, anyone with ability to alter the config variable gains arbitrary code execution on the server.
WTF?!?!? This is from a commit for I ticket I had made. How is this being removed here?!?!?
I cleaned this stuff in https://github.com/racker/virgo/pull/136
What are the "to-do items"?
Perhaps we should pull in a community sensu check plugin for the example, such as https://github.com/sonian/sensu/blob/master/dist/chef/data_bags/sensu/config.json#L18
This specifically is a silly change, but its easy enough to glance past in inspecting the diff.
Not sure if there is a standard way of documenting this. I'll look into this.
:)... another use of Y.mojito.loadSync(name, path);
It's a good idea, but this is not the right place to validate the user's dimensions.json, since this code is called several times, leading to multiple reports of the same error.
All YUI modules share a common base URL. In this case, yuiPrefix replaces that base URL. Once on user agent, the YUI base URL can be recreated.
Hey,

I'd like to avoid having the pattern "curl (non SSL URL) | sudo sh" in installation -- that's a huge security vulnerability, and basically giving full root access to anyone who can hijack the DNS or man-in-the-middle.

But even better: npm ships with node these days, so it doesn't need to be a separate install!  Please remove the curl npm bits here and below and I'll pull this.
а куда? садистик сказал что после истечения срока/показов переводить статус в удаленный. вот перед обновлением списка баннеров чистятся истекшие
Yes, if you want to use the bit setters/getters in the class you can use PotionBit.AWKWARD.getBit()
Plus, I usually add getters if there's a setter. Good practice in case myself or others need it at one point or another.
Changed to allow any colname types. I think the recycling and conversion already works because that's handled by assign().

```jl
julia> DataFrame({"a" => [1:5], "b" => pi, "d" => "X"})
      a d                 b
[1,]  1 X 3.141592653589793
[2,]  2 X 3.141592653589793
[3,]  3 X 3.141592653589793
[4,]  4 X 3.141592653589793
[5,]  5 X 3.141592653589793


julia> DataFrame({"a" => [1:5], "b" => pi, "d" => "X"}) | dump
DataFrame{ASCIIString}  5 observations of 3 variables
  a: DataVec{Int64}(5) [1,2,3,4]
  d: DataVec{ASCIIString}(5) ["X","X","X","X"]
  b: DataVec{Float64}(5) [3.141592653589793,3.141592653589793,3.141592653589793,3.141592653589793]
```
Use `Equals` with a string comparison option
If you use core.autocrlf=input in git, it will do the right thing wrt
newlines.


On Mon, Dec 17, 2012 at 10:34 AM, Matthias Mailänder <
notifications@github.com> wrote:

> In mods/ra/rules/system.yaml:
>
> > @@ -1,449 +1,479 @@
>
> might need a dos2unix
>
> —
>
> Reply to this email directly or view it on GitHub<https://github.com/OpenRA/OpenRA/pull/2518/files#r2430697>.
>
>
>
I remember you were talking about this... but the logic used seems unnecessarily complex
```
if(!(!Ext.getCmp('patientPrimaryContact').value || Ext.getCmp('patientPrimaryContact').value == ""))
```
I believe the below is sufficient to check that it's not "", null, undefined, NaN. Is there another case you're worried about?
```
if( Ext.getCmp('patientPrimaryContact').value )
```
Can uncomment
I don't see anything that verifies a timeout happens or a "connection error" is shown on timeout
done

Made an enum in util.js
I think using update is preferable e.g.
```python
for coord in filter(None, self.dependencies.itervalues()):
    derived_dims.update(coord_dims_func(coord))
```
// DROP getPartioningKey -- No longer called, leaving m_partitionValue unused.
yes the regular one is good here
Why is this a member, and not an argument to the one function that uses it?
I don't think that we want the ClientInterface to "remember" this value between function calls.
`PATH.join()` already does normalization of `./` and `../`
frag_state
On 05/11/12 19:36, Eyal Lotem wrote:
> In cabal-install/Distribution/Client/Config.hs:
>
>>          unlines (map (showPWarning configFile) ws)
>>        return conf
>>      Just (ParseFailed err) -> do
>>        let (line, msg) = locatedErrorMsg err
>>        warn verbosity $
>>            "Error parsing config file " ++ configFile
>> -        ++ maybe "" (\n -> ":" ++ show n) line ++ ":\n" ++ msg
>> -      warn verbosity $ "Using default configuration."
>> +        ++ maybe "" ((':' :) . show) line ++ ":\n" ++ msg
>
> I am just as used to point-free as I am to point-ful. For me it is just
> as clear, except the point-free one produces less hlint warnings :)

Maybe hlint is complaining about the use of ++ instead of :, rather than 
the pointful style?  I definitely agree with Johan, the point-free 
version is less readable, because it involves doing an extra reduction 
step in your head.
Duplicate of line 14!
So this bug doesn't exist in Opera Next, presumably?
Additional checks accepted.
Ah, this should be pdf.
Couldn't the META.json file be generated if configure is called by pgxn? Or is META.json necessary in the zip file? In that case, it might only be necessary when deployed to pgxn?

Bottom line: I am wondering if we can get rid of this file in the "normal" repository.
I think we don't need to distinguish loginResultForState between oauth1 and oauth2...?

Specifically, could this be simplified to just Meteor.accounts.oauth rather than Meteor.accounts['oauth' + setupOptions.version]?
Oops, read that wrong, should read "but does not work on win32". :S
`__hash__` and `__eq__` only using URI is OK with me. Please remove the comments.
Yes, I'm not really fond of some ways stuff is generated by CS, like this particular one :)
Same comment as for ``_createNewItem()`` above.
No magic numbers in the code please; reference a "static" variable containing the allowed permissions levels.
Use the following code instead:

``` ruby
name = controller_name.titleize
```
I angsted over that for about two hours, and then gave up and moved on.  I highly encourage something other than "parent" that means something-back-up-the-chain.
Please take care of proper indentation here.
DANGER: Possibility of Cross Site Scripting attack. For example, someone may be able to upload a file called that contain malicious JavaScript in its filename. The file name must be escaped before being inserted into the HTML.
email address  variable needs to be URL encoded
Can you use `Time.now + 86400` and add a comment to show why this number?
Again, the indentation looks off.
I kept the results as close to the prior script's results as possible, except og namespaced properties now include "og:' such as 'og:title'.
Good idea. I removed that module and moved the functionality to brackets.js

Now SidebarView.js and ProjectManager.js listen for the htmlContentLoadComplete event since they both init jquery vars on load.

From: Jason San Jose [mailto:notifications@github.com]
Sent: Tuesday, August 14, 2012 10:33 AM
To: adobe/brackets
Cc: Ty Voliter
Subject: Re: [brackets] Combine jdiel's "language switching" pull request with tvoliter's templatization of index.html using mustache (#1358)


In src/brackets.js:

> @@ -50,6 +57,12 @@ define(function (require, exports, module) {

>      require("thirdparty/path-utils/path-utils.min");

>      require("thirdparty/smart-auto-complete/jquery.smart_autocomplete");

>

> +    // Load HTML Content

> +    // The htmlContentLoad module renders all of html in Bracktes.

> +    // If a module depends on the DOM being fulling formed before it is loaded

> +    // then the module should listen for the event "htmlContentLoadComplete"

> +    require("htmlContent/htmlContentLoad");

Should have seen this sooner, but do we need a completely separate module anymore? htmlContentLoad.js won't be reused anywhere else. It seems like we could inline the Mustache.render() call inside _onReady() in brackets.js.

—
Reply to this email directly or view it on GitHub<https://github.com/adobe/brackets/pull/1358/files#r1374771>.
Agreed, if we don't need globals, then we should use them!
It's still under debate whether we should be using a python library that wraps a lot of the elasticsearch functionality - this current implementation (both yedi's and mine) we attempted to do it with pure ES api access with json.
COUNTRY_PATTERN
Can this be indeed removed?
It may be nice to quickly explain the behaviour of this class, as this is not trivial (even if not overcomplicated).
Maybe one author signature can be passed for both the author and committer? Can't think of a case where they might be different

```obj.remove_note(author, ref)```
ad 1:
I cannot understand how RegistrationView with form_class = forms.RegistrationForm is doing all this:

         form = forms.RegistrationForm(request.POST)
         if form.is_valid():
             username, password = form.save()
             new_user = auth.authenticate(username=username, password=password)
             auth.login(request, new_user)

ad 2: thanks for the tip

ad 3: Yeah with that kind of implementation (the kind I could not get to work with ME 0.6.2(?)) it is all clear. I was referring that question to implementation when there is a separate class Foo(Document) that stores additional user data but my question is now obsolete.
I have a problem with this, I can't figure out where to put it, also I couldn't find any usage of this method on nodewacher.
Ah scratch, that I've used `Class.new ...` instead of it's working fine!
If I understand correctly, our DOM Fragment code can grab this style tag and put it in head/ remove it when needed.
Multi-line comment block
I don't know `rbenv` very well, does this default to the global version if you're not in a specific version?
This might leak depending on the type returned (e.g. BSTR). Please use a CComVariant here (works like a safepointer and releases resources in its destructor)
Tutaj też comments wywalić
We should choose another directive name, since there's no such a word.
格式，void换行