labofsahil/github-event-dataset-2012 huggingface dataset:
Fixed pom plugin configuration

Pom's plugin configuration was broken
CAPEDWARF-15 Proper XMPP implementation
new copy. better js.

added more information in the hero unit.
allowed the links to fail gracefully.
Twitter oauth #24

Enables and tests twitter 3-legged OAuth using JSOAuth. @replies users if track queued or not found.
Pass instance to processors

That would allows to store or use instance values in processor and also close #11.
Get arguments

I recently did a pull request that would've broken everything. Turns out I mixed old and new code. This is the code as I use it myself now. I tried catching GET parameters by ending with "**" but that didn't catch behind the "?", now choreographer parses the GET parameters (when present) and adds them as a last parameter.

Eg: router.get('/register/*/*', function(req, res, username, password){ console.log(username + ':' + password); });
or   router.get('/register', function(req, res, args){ res.end(args.username + ':' + args.password); });
or   router.get('/register/*', function(req, res, lobby, args) { res.end('Entered lobby ' + lobby + 'arguments: ' + JSON.stringify(args));} );

As you can see it's fairly elegant. Someone who doesn't know it's there doesn't notice it. It allows slightly more "standard compliant" behavior of putting arguments behind the "?".
EOFError (bad content body) with special characters (maybe only on ruby 1.9.2)

when I submit a query with a file and with another params which contains special characters (ie: éèîü etc ..) it breaks. 

EOFError (bad content body)

I don't know if it's the right way to fix it but it works !

Note: tested only on ruby 1.9.2
More understandable text in options

I does some work on text in popup and options page.
Add ability to specify default queue name. 

In my case I needed each application server running the same code to use a different queue name. Very useful.
Fixed cylinder primitive

@ Jerome
Just thought I'd help get you started :)
1.7.4 mulitconfigure and p264 codec support implemented

Removed the c# implemention and extracted the decoding from the C SDK and compiled that under Managed C++
wired up get token, fixed exceptions on bad request to getregistrationht...

...ml
Added random Code I dont want lol

Just delete this
Timob 7517: Android: Simple animation is not working in Android

http://jira.appcelerator.org/browse/TIMOB-7517
Test case is in the ticket.
Please also do a KS sanity check and run drillbit test for UI module on both runtime.
Submitted a new version of puzzler 26 as puzzler 28

Based on the [original submission](https://github.com/scalapuzzlers/scalapuzzlers.github.com/pull/55) by Vassil Dichev
Fibonnaci Generator Added


Added fib_sum to requestHandler.js
Added /fib_sum to index.js

Fibonnaci sequence is accessed by adding /fib_sum?limit=# and has a hard limit of 1474 which is due to javascript's integer limit of 1x10^(300).  Can be overcome using large integer handling at request.
A talk about hacking with redis

I propose...
Modification to actionpolicy.rb

As per my discussion with R.I. in IRC a few days back -- this is a modification to actionpolicy.rb to enable setting a more complex default policy file.
LPS-25129
Fix sqlalchemy import

* pulse2: fix sqlalchemy exception path
update getOptionSet()
Fix a typo visible on the home page (MISSON -> MISSION).
Changes to rails3_acts_as_paranoid for Rails 3.1

I'm hacking around in rails3_acts_as_paranoid. In the process of making my own changes (which I hope to fill you in on later), I found the errors (mentioned [here](https://github.com/goncalossilva/rails3_acts_as_paranoid/issues/40)) caused by using the Rails 3.1 version of ActiveRecord... and managed to fix them.

There's only 3 changes of real importance:

* Rename the call in uniqueness_without_deleted.rb from mount_sql_and_params to build_relation
* In the same call, pass an Arel::Table instead of a String table name.
* Remove the aliasing of primary_key_name to foreign_key in ActiveRecord::Reflection::AssociationReflection

Feel free to ignore/revert my changes to the Gemspec; those are mostly for my own internal purpose.

Hope this helps,

Craig
Project settings improvements

As discussed here are the two changes.
whos the troll ?
Damn it. Sorry, Martin, I haven't noticed. I skipped the 'branch' line in the commit notification e-mail by mistake and apparently the github interface doesn't give any clue on the branch of the commit(s). All I can see is a large "elua/elua" on the top and no reference to Mizar32 at all on the rest of the page.
If you do this you will not have the fully qualified hostname !

Then the gateway will contains urls like http://mkwalski/agents ans some host may not know how to connect to it .
I thought about it, but I suspect not.
well, if you manage to have a picture in the root of the picture window ;)

path is empty.
We had about 100 warnings (and most of them not serious) when compiling R15B which made it hard see new serious warnings. Thats why I would like to remove and preferable fix those old warnings.

I'll make a habit of not disagreeing with you when you are right. =)
抓了有什么用吗。。
I'd recommend running through these:
https://wiki.jasig.org/display/CAS/CAS+Functional+Tests
(minus the ones that do ticket validation since we don't have that in the flow).

We should probably automate these at some point ;-)  These tests though should make sure the major components work (Gateway, renew, warn, etc.)
HTML5 parser also lists "command", "embed", "keygen", "wbr", "param", "source", "track", "col"  as non-deprecated self-closing tags.
$value = iterator_to_array($value, false); // solved bad behaviour for items with same key
I would apply the boy scout rule when there are major changes not minor tweaks like this.
@carlosantoniodasilva @vijaydev  thanks !
Would be great to have the reason of reverts in commit message :)
You can change it yourself on installgentoo.net if you want.
Commit your changes to your local repo, then you can update from github as usual without losing your modifications.

And Eksopl can always change it later if people start complaining too much.
This is a different area, remove current hightlights, then fine the next.

somehow, we could estimate the time cost to remove current highlights. it should take longer than (total / 100) * 25 ms to remove them.

So we would only find the next when all of those highlights removed.
Instead of the change you made to the Hello World examples, can you append "This example assumes you have setup URL rewriting with your web server (see below)." to the existing paragraph above each Hello World code block?

Keep the fix you added to the Documentation section. I'll likely merge this into the `develop` branch and launch these changes with version 1.6.0 in a few weeks.

Thanks!
Another way to think about it is that a rule is a command saying that the target should be made when its dependancies are updated. An ordered dependency is a fancy way of making sure that a chain of these rules is executed in the right order. A phony target is a one which does not correspond to an actual file, and so should generally never depend on a file. How can a non-file depend on a file? Remember that depend here doesn't mean use, it means that the rule should be run when the dependancies are newer than the target. But if the target isn't a file, that becomes meaningless. Your phony target uses the Makefile, but this isn't a dependency in the language of Make.
Found Gullerud patch just now:

@@ -608,6 +608,8 @@ proc run_commands { prompt command } {
       set command "set logging session disable;$command"
   } else {
       send "terminal length 0\r"
+            # Also attempt to turn off timestamps in newer IOS
+            send "terminal no exec prompt timestamp\r"
   }
   # match cisco config mode prompts too, such as router(config-if)#,
   # but catalyst does not change in this fashion.
The performance of the structured dtype cache isn't interesting because it's a cache-- I only think we might want to do that for small indexes
Oleg,
First of all thanks for this work. I will not merge it direct.
This way I will see for some other problems - i.e I will add this line by line.
Cannot compile after this commit:
g++ -o bindings/python/mapnik_python.os -c -DHAVE_JPEG -DBOOST_REGEX_HAS_ICU -ansi -Wall -pthread -ftemplate-depth-300 -DLINUX -DBOOST_SPIRIT_THREADSAFE -DMAPNIK_THREADSAFE -O3 -finline-functions -Wno-inline -Wno-parentheses -Wno-char-subscripts -DNDEBUG -DHAVE_CAIRO -DHAVE_PYCAIRO -fPIC -Ideps/agg/include -I. -Iinclude -I/usr/local/include -I/usr/include -I/usr/include/freetype2 -I/usr/include/libxml2 -I/usr/include/gdal -I/usr/include/postgresql -I/usr/include/python2.6 -I/usr/include/cairomm-1.0 -I/usr/lib/cairomm-1.0/include -I/usr/include/cairo -I/usr/include/sigc++-2.0 -I/usr/lib/sigc++-2.0/include -I/usr/include/pixman-1 -I/usr/include/directfb -I/usr/include/libpng12 -I/usr/include/pycairo bindings/python/mapnik_python.cpp
bindings/python/mapnik_python.cpp:98: error: âbool mapnik::python_thread::thread_supportâ is not a static member of âclass mapnik::python_threadâ
scons: *** [bindings/python/mapnik_python.os] Error 1
scons: building terminated because of errors.
make: *** [mapnik] Error 2
Dangerous to push below stack pointer. An interrupt coming here will trash the value on the stack. I think you need to adjust sp before this store.
Thank you for reading and commenting. Rereading the docs again I see where I got confused. A very common use for us is to bind and execute a single query. I think I will take a look at extending the pgsql:equery path to support such single query execution, but with reuse of a prepared query.
m_iConnecTimeout is already in millis
Here is the benchmarking test code I used: https://gist.github.com/1959040
Here are the results: https://gist.github.com/1959092

Incrementally small "improvement" on fetching single item. Incrementally "worse" taking small-medium sized sub-set.
The rand() size of the resulting set may skew the results for the third test set but they seem to be fairly close.

Your call, of course, but it doesn't appear to do any real damage. "Real world" usage may dictate otherwise. :)
@richarddong 这个partial是循环调用的，写在里面的所有东西都会随每一个条目render一次
You might want to get rid of the floppy comment here, or update the value of the var to be rhel58-64
As ugly as aliases are, we should consider having `enqueue` as an alias for `unshift`. That is clearly a standard computer science term. And this is the primary thing to use if you want a queue.
`u8_strwidth` gives a fast implementation of this that works for both ascii and utf8.
But it is done with every AddPath() atm anyway, right?
No worries. The API should be 100% backwards compatible with the current `node-fork@0.3.0` except that we also expose `require('node-fork).shim` and `require('node-fork).native`
As a proof, try opening [this build page](http://travis-ci.org/#!/gildegoma/salat). It responds with 503 because rendering 50 MB JSON files causes travis server to go above Heroku RAM limit (500 MB).
:)

The code is still a bit ugly, and I'm sure there are some bugs lurking. But yeah - it should handle reconnection properly now, which is kind of important.
in this line we are out of array
BattlegroundSA.cpp line 454

     m_Gyd[SA_EVENT_ADD_GRAVE_A] = ((GetDefender() == ALLIANCE) ? BG_SA_GRAVE_STATUS_ALLY_CONTESTED : BG_SA_GRAVE_STATUS_HORDE_CONTESTED);

m_Gyd length is 3, but SA_EVENT_ADD_GRAVE_A = 3. out of index
What is a good fix?
it says Player::SetClientControl(Unit* target, uint8 allowMove)   , so not sure why would 1 prevent moving.
use git_id before pushing and you'll never forget :)
The need to be different, they are two separate themeSets... We don't want changes to one to be reflected in the other
this shouldn't be localized
I feel so dumb right now. I will fix the issue and re send you the updated pull request.

Thanks,
That's the default, so just AddonManager() works.
That's an interesting point. We added that to resolve the rounding issue that you'll see - basically, the last column won't necessarily be aligned right with the grid depending on which browser you're in. In theory you could resolve this by using ten column instead of three and putting a nested row inside that to limit the width of the content. Not sure if that's better than having ragged column edges or not. Thoughts?
Can you please document the parameter (same below)
yeah, true. I think I have an idea for how to get rid of it.
The second two modifications look good but the first one.. casting from int64 to double, not sure if I misread something here but I don't believe that would be an appropriate work-around.

I won't know until I check further whether the error represents a more serious error with my code, a VS bug or what. Can you open the pull request against my personal version of the repository?

I doubt I'll have time to work on this at work for a while.
ここは、基本 ememcached_app の起動時にやるべき。 sup あげて、そこにぶら下げる感じ。
Doesn't the shell eat the single quotes before they make it into mingw's gcc?
actually there should be some error message

> version is a required argument. exiting...
Nice, hacking away on cljs! Love this. I'm interested to hear your feedback on the Clojurescript one framework as well.
CPP     xbmc/cores/paplayer/SIDCodec.o
CPP     xbmc/cores/paplayer/TimidityCodec.o
CPP     xbmc/cores/paplayer/VGMCodec.o
PAPlayer.cpp: In member function ‘bool PAPlayer::CreateStream(int, unsigned int, unsigned int, unsigned int, CStdString)’:
PAPlayer.cpp:507:5: error: no matching function for call to ‘CAudioRendererFactory::Create(NULL, unsigned int&, PCMChannels*&, unsigned int&, unsigned int&, bool, bool, bool, bool)’
PAPlayer.cpp:507:5: note: candidate is:
../AudioRenderers/AudioRendererFactory.h:34:26: note: static IAudioRenderer* CAudioRendererFactory::Create(IAudioCallback*, int, PCMChannels*, unsigned int, unsigned int, bool, bool, IAudioRenderer::EEncoded, bool)
../AudioRenderers/AudioRendererFactory.h:34:26: note:   no known conversion for argument 8 from ‘bool’ to ‘IAudioRenderer::EEncoded’
make[1]: *** [PAPlayer.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [xbmc/cores/paplayer/paplayer.a] Error 2
@ozten - that is intentional.  A value is only returned when there is a problem that must be addressed.  So, returning "LOCALSTORAGE" means the reason that BrowserID is not able to operate is because the user does not have localStorage.  Now that I have added LOCALSTORAGE_DISABLED, the original names are not as clear.  I will update these.
this is the same guid as Windows Media Player's?
Hey there, I've been experimenting with using Swig in my project Bound, a static blog generator, and also on an internal project at my company. I noticed that the file you generate for the browser leaks a bunch of stuff into the global scope. When you sed the requires out with this line, you are actually removing the 'var' from the variable declaration are of some files (at least, this is true for index.js), which results in JavaScript attaching the values as properties on the window.

Are you presently working on a revised script to generate the browser-side library? If not, would you be willing to consider a pull-request with a revised solution to the problem?

Also, would you be willing to consider exposing a hook or other mechanism to allow for adding new filters when using Swig in the browser without pre-compiling them into the library?

Incidentally, the fact that you are leaking certain values into the global scope allows us to add filters while running in the browser. A formalized version of this behavior would be nice to have ;)
Ok. I understand.
= =Icon基本会搬ACE和XP的，支持图层是必须的TUT
It appears that the proposition is no longer making it into the google analytics custom variable. I wonder what has changed.
looks like we'll all be fans of ruby 1.9 with rails 4. 

Thanks!
@sferik 
There it is https://github.com/sferik/rails_admin/pull/904
Specify here whether it looks into the PATH (I think yes, but it's just implied from the example).
1. 咦那为什么注释掉了我以为不用了呢
2. 放在 config/initializers/ 的脚本每次 Rails 启动都会被执行的，不管在什么环境下，所以感觉还是放到 seed.rb 里面比较好
3. find_or_create_guest 我 Google 过，唯一的记录是 Homeland 和 Ruby-China :-)
What's the sleep for? It looks harmless enough, but I'm curious.
I think that doesn't work for buttons?
No falta un espacio para que se renderice correctamente?
created because site works in development locally, but not on heroku. this didn't work. do not use, reverting to finished
stub pages for Noatta server and client demos are built.
client check-in sort of works with identify button (can't seem to get it to launch identify inline on page load). Once user is identified, does not create new session... just sends alert for userID
M2Sys ids are manually added/removed to database.mdb via DBTest.html
Whoa - looks like I never thanked you for these calebr. Thanks!
Hi. Can I merge your port with mine?
> Type Belar
> There is only one item, but suggestions' box height can feet 4 items.
> Also I see scroll disabled. Do we need to show it users?
Fixed this:
https://github.com/MblKiTA/js-trainings/commit/3077f68b10f4a3d81fbb7e0cf7d9c01346ff3b0f

Thanks
2012/1/12 Cheng Sun
<reply@reply.github.com>:
> Is there any advantage to this getter/setter pattern over just using the variables directly?
1. I need class to have pointer to in common/win32_fullscreen.h (I
will use it in ActiveX too), or I will need have multiple pointers to
properties in my classes (it's ugly way)...
2. In set/get we can make some additional processing (for example
notification of property changing)


>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/RSATom/gtk-npapi-vlc/commit/f920a5c6b61bfc5f74514ebf31318e4b44e827e0#commitcomment-858071
Properties should get some better names
How do you reproduce ?
What's this if not accounting for the number of days in a month?
What happens if you remove everything except the off-by-one fix?
ranní pullRequest https://github.com/Clevisaci/HttpPHPUnit/pull/9
I may be mistaking, but I think that the if added in js/functions.js introduces a problem:
 some_database ->some_table -> SQL tab -> press go on the "Run SQL query/queries on database some_database" form
 nothing will happen, the form will not be submitted.

reason:
 in js/sql.js, function: $("#sqlqueryform.ajax").live('submit', function(event) 
 the following code will return false, and cause the form not to submit
176      var $form = $(this);
177      if (! checkSqlQuery($form[0])) {
178           return false;
179      }

the checkSqlQuery(theForm) will fail after the above mentioned if.
Instead of setting the `results.length = n` you could assign `results = Array(length)` and make a var `length = this.length >>> 0`.
You are right - I went on a run and decided I'll update the TODO to add samarai or another gem to encrypt credit cards. This will fully support refunds, also.

You are welcome to help out. Note the TODO, the highest priorities in my mind are:
1) Testing (that's a pretty obvious duh) - since this was built out as a prototype, TDD wasn't necessarily a client-supported venture :) And also once I get started with a test framework, I can write tests, but it's getting started that's the hard part for me.
2) CC encryption + fix refund support
3) Helper method for Google Analytics tagging (pretty simple)

I guess I'd just note that this is trying to be simple, and not solve every problem. My belief is that modularity / engines can introduce additional functionality, but that it shouldn't be included the core. 

Comments are appreciated.

Yes, I did work on shoedazzle, as an End Point employee consulting for Rails Dog.
Jolly good commit
Awesome spot indeed, will update shortly.
i like it))
Huh?

`assert !(Word.zero > Word.one)....`
For better or worse ruby is 100% standardized on unix line endings and 2 spaces for indentation. SciTE on windows has options for setting those and it should still work just fine.
You don't like random whitespace? :)
Vezi cu Cornel, cred că ar trebui să intre în meniul "actions", care a apărut între timp pe branch-ul master.
That's two... I assume the third range is those rows not in either of the two above? And I also assume the ranges can't overlap ? Comment could be clearer...
I agree, but does Django provide an elegant way to subclass User that wont require further changes to how users are accessed? I can't seem to find one.
will it be better to save settings before exit?
Checking out comments
When the property is set to "true" it will not report any Open GL errors in the console.
When the property is set to "True" it will report basic Open GL activity / errors.
I can't confirm this. Works with space and shift for me.
The text for the keymap is from native Javas ``KeyEvent.getKeyText()`` so it's probably a java translation missing for your language or sth...
+1, muy subnormales tendríamos que ser, la duda ofende
sorry, will do.
I agree with both of you. Seeing it side-by-side made a difference.
I basically attempted to serialise everything that has to do with the
current game state. This included all the generated map data and monster
positions, so it kinda exceeded the bundle size limit for android. So to
answer your question, yeah it doesn't save properly.

On 8 March 2012 10:58, Pigeoncraft <
reply@reply.github.com
> wrote:

> what was wrong with this code does it not save properly?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/shawntan/minicraft-android/commit/0dc725b5def8941b51742f6cb9a7ecc98495c175#commitcomment-1056816
>
Зачем тут проверка на obj.overlay, если после предыдущей строчки это уже верно?
Whats the point of this when the line above it does the same thing?
agree, moved to botom
This works as far as I've tested it with android emulator.
Not sure how it works with NETWORK_PROVIDER as android emulator supports only GPS.
сокращение input -> inp экономит два символа и тратит 2 секунды на понимание, что же значит inp_command. Не вижу выгоды.
Относится и к comand->com снизу. Питон делает упор на читаемость кода, не стоит экономить символы.
[#APPTENTIVE-49]
Don't know why, but it was the only file I found in the project using windows EOL format.
It should fix those strange pull requests changing the header. :P
import android.net.NetworkInfo, unbreak these lines.
yo creo que con solo striptags no recogera los &lt; (a menos que rigor potencier lo tenga hecho asi internamente), lo cambio según vea para que sea riguroso ok? :D
>inside threads
Aah. I was look from board. That's I fool, comrade.
Btw, that stuff is possible to modify for other imgboards.
I think you need to write a white paper on best practices for `composer.json` files :)

Fixed: 9c4ac4a92ccbab4fd713d4225401268873280626
Preferirei qui si usasse la sintassi:

    from foods import (spam, spam, spam,
                       spam, spam)

Invece del `\`
I am guessing it's added twice because the element is already of `type="submit"` and it's set again. Just super weird that Zend_Form seems unable to merge attributes and produces _broken_ markup. That seems to be the real issue. ;(
Why this format and not 'If key in dict' used elsewhere?
Du coup, même remarque (j'ai regardé les commits chronologiquement), ce serait plutôt attrapé (donc "attrape" sans caractères accentués), non ?
Inconsistent quotes... try to stick with single quotes as that seems to be your standard throughout the rest of your pull request.
Haven't fully wrapped my mind around this one yet, but this workaround smells bad. Can't we fix the actual problem?
it's not technically related to this change, but we need an upgrade test in general, that confirms *new* dependencies found during an upgrade get installed.
I see two uses for the `DEFAULT_USERNAME_ATTRIBUTE` constant:

* Having that value as part of the list of username attributes allows the service registry to better indicate the CAS behavior when no attr is selected. By including for instance `(default)` in the username column, I think it visually better represents the configuration of a given service as opposed to just having blank in there, which may seem that the service is not returning anything at all for its username!

* More importantly, the constant allows you to switch over and reset the username attribute back to default. Suppose that in svc registry, a given service is configured to use `someAttribute` as the username. If the constant is removed, when the deployer decides at some point to remove that rule and reset the attribute back to the default, there would be no way of configuring that (as the selection of username attributes in the UI is noted by radiobuttons). So, having the default constant in there along with the rest of the available username attributes allows one to revert the setting, if needed.
It is kinda ugly, but it is backend code, the fact that you have to know the side effect of the accessor does not bother me much especially considering the code of the accessor is actually just above. Also, those line are required only because we have to copy the data back from device for PETSC to use it, this approach being inefficient, i assume that we will use an OpenCL solver in the future, then those lines will no longer be required. Considering this I would prefer leaving it like that in the meantime than changing the user API.
ok. Agree :)
No there isn't currently.  Should I look to put something into akka.performance.microbench?  Need some pointers here.
Just discussed with Viktor: the number of parentheses is too high, so I am thinking about making the syntax a bit nicer. Don’t let that get into your way, I can do that after you merged this and then I’ll experiment with how different options look like.
ok, as long as it's still possible to to implement subclasses in java also
like your boss!
So you have made sure that in all places where exceptions can be thrown, the actor instance is not corrupt?
protected[akka], or prevent subclassing by user code
no, no, I'll go through all comments and fix all of them
for consistency with `DESTDIR` I think you should remove the underscore in `DATA_DIR` (also it's now based on `PREFIX`)
No need for this any more, the latest master registers classes that are inherited from `Widget` automatically and as all layouts and widgets are inherited from Widget...
I ran 'python2 setup.py build_factory' to get this, btw.
either htsql needs to be a requirement of ckan or if optional then we need to test the import and act accordingly - not knowing what the plan is with this I'm not sure which is the correct solution
@icmurray Yeah this is deliberate, I don't want large dicts (that might contain the dicts of all the group's packages recursively, etc.) in the activity streams table, just the basics. All the *_acitivity_create() functions do this. Activities get added to this table all the time and never deleted from it!
Perhaps mention at a high level what the event body will contain. No need to go to crazy detail, but things like indicating that the *.finish events will carry whether or not the operation was successful.
gettext on all of these.
This function shouldn't return twice.
is it weird that pending returns a card?
Likewise with the JSON. You should be defining *schemas*, not *file formats.*
There is not. Perhaps there should be one. What would be the gain, however? Or would it let us subclass for various types?
I don't think this really works right now, but leave it there
We'll need a config example for this in pyhole.utils.
Or even better, "dark background" and "light background" depending on current state

On Sep 13, 2012, at 5:40 PM, Andrew Chen <notifications@github.com> wrote:

> In res/values/strings.xml:
> 
> > @@ -163,6 +163,7 @@ http://k9mail.googlecode.com/
> >      <string name="show_full_header_action">Show full header</string>
> >      <string name="hide_full_header_action">Hide full header</string>
> >      <string name="select_text_action">Select text</string>
> > +    <string name="toggle_theme">Toggle message colors</string>
> How about "Toggle background color"?
> 
> —
> Reply to this email directly or view it on GitHub.
>
cool Thanks
Interesting, no clue what's supposed to be there. Will check tomorrow.
this needs to be a `def`
(Oh dear, sweet irony).  Perhaps in the interests of documenting the class, you could replace this with a short description of TUIButton.
Let's keep the brace style consistent with the rest of the Java code in libpd.
OK, I think I understand. The d3.transform call canonicalizes the result of interpolation (e.g., mapping rotate(780) to rotate(60)). Should be fine either way.
I've fixed a similar issue for week-of-year/day-of-week parsing too.
should be description_len
Got it. See me comment about deprecation.
Hehe :), thanks.
Ok, no problem. :)
All the cases needs `belongs_to` in the model, so I think is better to choose one of then.
Well, actually, after looking into Typers more closely, it seems that default arguments take precedence over tuple applications, so augmenting `checkNotMacro` might lead to unexpected results.
This will write
  "... Is class com.squareup.FooEvent registered?"
would you prefer
  "... Is com.squareup.FooEvent registered?"

(either works for me, but the toString on Class is often surprising)
In general I prefer brackets except for special error-handling cases like this where it does not contribute to the logic flow of the code, rather it's essentially an error-out in the way an exception would be. Being somewhat boiler-plate, I like to keep it short and out of the way.


More interesting are uses like this, where I feel it could go either way, but I felt the former was more readable, but you may disagree?

  if (qf->last->position < qf->first->position) {
    uint32_t newLastPosition =
        qf->fileLength + qf->last->position - QueueFile_HEADER_LENGTH;
    if (!QueueFile_writeHeader(qf, newLength, qf->elementCount,
        qf->first->position, newLastPosition)) return false;
    if(!freeAndAssignNonNull(&qf->last,
        Element_new(newLastPosition, qf->last->length))) return false;
  } else {
    if(!QueueFile_writeHeader(qf, newLength, qf->elementCount,
        qf->first->position, qf->last->position)) return false;
  }

against the expanded version:

  if (qf->last->position < qf->first->position) {
    uint32_t newLastPosition =
        qf->fileLength + qf->last->position - QueueFile_HEADER_LENGTH;
    if (!QueueFile_writeHeader(qf, newLength, qf->elementCount,
        qf->first->position, newLastPosition)) {
      return false;
    }
    if(!freeAndAssignNonNull(&qf->last,
        Element_new(newLastPosition, qf->last->length))) {
      return false;
    }
  } else {
    if(!QueueFile_writeHeader(qf, newLength, qf->elementCount,
        qf->first->position, qf->last->position)) {
      return false;
    }
  }
You also need tests when y is odd or even.
The problem is that "anticommutes" doesn't really mean "not commutes", so commutes=False for it is a little confusing for it (the former implies the latter, but not the reverse).  A way would be to just have both "commutes" and "anticommutes", and have it be an error if they are both true (or it implies that it is zero or something).  Probably the assumptions system could be leveraged somehow to do this automatically.  Ditto for symmetric/antisymmetric.
Replace this with 

```py
print "%s could not be loaded due to %s." % (module_path, repr(a))
```

Aside from the fact that using string formatting is a better way to build up a string, this will give the exception itself, and not just the message.
Not sure I understand which alternative you are suggesting.

I suppose the line is not strictly necessary, but we do want rational multiples of pi to fall through, so we cannot just use an else.  The default fall-through for everything should  be to return un-evaluated.  But this function is basically a switch statement where we prune off special cases as we can until until evaluation is  complete (if possible).  This is a clear corner-case, where special treatment   seems appropriate, so I would be confused reading the code if we didn't  explicitly address it.  Also, there's a lengthy comment at line 407 about our current uncertainty on how to evaluate cos(oo).  I can duplicate this comment if people feel that would be helpful.
This must have been from when I was debugging the SparseMatrix issue. Those two lines should be deleted.
the Kane method class isn't called KanesMethod, so why do that here? It's just more to type. There are no other naming conflicts in this module are there?
Yeah I just found it through googling as well...

On 05.04.2012 17:13, Christopher Smith wrote:
>> @@ -1438,3 +1438,19 @@ def pretty_print(expr, **settings):
>>       print pretty(expr, **settings)
>>
>>   pprint = pretty_print
>> +
>> +def pager_print(expr, **settings):
>> +    """Prints expr using the pager,  in pretty form.
>> +
>> +    This invokes a pager command using pydoc. Lines are not warpped
>> +    automatically. This routine is meant to be used with a pager that allows
>> +    sideways scrolling, like ``less -S``.
>> +
>> +    Parameters are the same as for ``pretty_print``. If you wish to wrap lines,
>> +    pass ``num_columns=None`` to auto-detect the width of the terminal.
>> +
>> +    """
>> +    from pydoc import pager
>
> I see this is part of the std lib, so I guess this is ok...I've never used it, however.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/1192/files#r650940
>
I've removed the part of code that corresponfds to `plot_beam` function
Hm, I just added an additional parameter for `send304()` holding a fallback value in my local repo. I feel that's the best method ...
This method is four space indented.
To say the truth.. I was a bit lazy... i just saw all the actions there so I just put even that one...

refactoring in few sec...
excepton
It could be right depending on routing.
- parenthetical expressions like this need to be part of a sentence.

- I think, in retrospect that "for points on earth," is redundant if you say "latitude and longitude."

- in general, we should probably strive to avoid using parenthetical like this as much as possible. and I think this example can be combined into the sentence.
ahem :) anyway, you get the idea.. just find these and remove them
@imkingdavid -,-. I think GitHub had a quirk here. I replied _(and it showed)_ to that one. But yes, I replied to @igorw
I instantiated the object within the append_sid test but it didn't recognize it as the object when globalized in append_sid itself when run in the test. The only way to get it to recognize it as an object was to define it in the global scope. I may have been doing something wrong though so I'll try again.
As far as I can see, you're not actually using `$i`?
I just said that (see my comment two lines down) ;)
@p-scratch can't fix it yet because my computer is on warranty repair (hdd died) since almost a week ago.
That seems to be more work than expected. All other variables and database entries refer to the topic title as 'subject'.  Even the user is displayed a 'subject' line in the posting page. So maybe I should just change the entries in the language files from title to subject. But as imkingdavid said, I think users will find 'title' more familiar. Can I have a final word on this?
This looks ok.
We need to destroy the _ext cache to get accurate enabled/disabled status, we need to destroy the _ext_finder cache so that all enabled extensions' services.yml files will be found, and we need to destroy the container cache so that the services defined by extensions are also included in the container.

Because one of the above is given as a variable, it doesn't really make sense to hardcode the other two in. Then again, thinking about it, it also doesn't make sense to not use the cache_name property at all by switching to the purge() method.

Would be be better to allow $this->cache_name to contain an array of cache items to destroy? For example:
```php
// In the __construct() method:
$this->cache_name = !is_array($cache_name) ? array($cache_name) : $cache_name;

// Now in each of the methods that require the cache to be destroyed:
foreach ($this->cache_name as $cache_name)
{
    $this->cache->destroy($cache_name);
}
```
Any reason to not make `$user` required and use dependency injection rather then the global. It appears that all calls to this constructor set the parameter anyway.
However this comment is not quite right. It should read:

Change style for users who use selected styles
This is writing to global $config which you probably shouldn't do. You probably should just get rid of all that legacy code and make sure database updater converts the data correctly.
Should not return by reference.
strictly speaking this function isn't so... ideal.

APR has a good example of the ideal function:

https://github.com/apache/apr/blob/trunk/file_io/unix/tempdir.c#L40

I think we can merge this for now, but we will want to come back to this.
cleanup aisle 5 with the rebuild function fcac8ef
Why remove this log line in this patch series?
I don't see any problem here, lost of software allows to build in root dir. Other tools like autotools allows to build in root dir. Can you give any drawbacks on a situation when we call cmake . in root dir?
```cpp
#undef _ISOC99_SOURCE
```
?
The dimensions.json file should contain an array, as per YCB specification.  (The issue is perhaps that we don't do any validation of the structure of a user-provided dimensions.json file.)
Can we use Util.dateTime(time, 24) here?
Please see my comment here (https://github.com/vikassingh/Raxa-JSS/commit/198a47c24e5de62c8290f4f741d2011b410cdca2) regarding refactoring this code.
Yes -- I ran into this issue as well. Have emailed the OpenMRS dev list to see if there is any way we can get around it.
remove $params['id']
You're right concerning b_row.
For Dz, you can simply loop over its elements, and zero them. There may be a simpler way, depending on what your Op is actually doing.
Anyway, you should launch the test with THEANO_FLAGS=mode=DebugMode,DebugMode.check_preallocated_output=ALL at least once. If it fails, that means that the Op is using output memory incorrectly.
I fix the %lld in printf code, VOLT_DEBUG code has not been runed. Thanks, I fixed it.
Does this removal relate to the rest of the pull request?
I introduced the HashSet member to keep track of the unique tables across the entire tree. In case of the recursion (sub-union expression) the parent content is passed to the child and then replaced back by the child set after the child parsing is completed. At that moment child set contains both parent and its own table names.
今installフォルダを再帰的にファイル削除するメソッドを書いています。完成したらコミットします。
Is this no longer needed now that a spiral type is being used?
this is wrong. DBAL switched to MIT
Thought about this. Not that hard to change.
Space here between brackets and argument.
don't ask me, it was there...
I have something that I haven't committed yet that could be added here.

```js
return matcher( elem, context ) && elem;
```
This function should move to the dt_preproc.sql_in, since this is specific to the preprocessing phase.
If you're not using 'id', use _.each instead of for/in (which gives you the value directly instead of the key).
thanks, but I get NotImplementedError ' gevent is only usable from a single thread' 
:-((((
`funfactory.helpers.urlparams` has a slightly different behavior than this function. This function allows you to pass many dictionaries, that will be considered in order, with the last one being the highest priority. Also it works indendent of urls, which is useful for me.

That being said, I'm a big fan of using other people's code, so I will see if I can use `funfactory.helpers.urlparams` instead.
Headed for bed, so I'll do an update tomorrow.

Thanks.
@jacknagel I'd like to know more. How to do a "Requirement"? I haven't found anything in the wiki.
Not needed anymore. Add a `depends_on :x11` instead.
Doing terminal replacement like this is sketchy - also the new HighLine instance has no access to any of the same variables (page_at, wrap_at, use_color) that the core Highline instance has.  Just call puts $stderr for now instead of super.
Perhaps we should define somewhere what `display name` means if we're going to use it in the documentation. While `display name` is what reddit uses internally, and thus exposed by this wrapper, someone new to reddit development may be confused by that.
Jasněže vždycky ne, ale hledáš takové kombo technologií, které dobře otestují a co nejmíň otravují. Update záznamu můžeš (vlastně musíš) otestovat i o úroveň výš, na tom Seleniu.

Podle mě není náhoda, že se unit testing tak rozmohl zrovna v době, kdy se přešlo od superstriktních a nudných jazyků typu Java k lehoučkým, akčním jazykům. Vypadá to jako psychologický mechanizmus: junior programátoři utíkají od striktních postupů, které dobře zaručují výslednou kvalitu, ale táhnou se; senior programátoři si z mládí ten pocit pamatují, takže zavrhují staré kvalitu-zaručující postupy, ale vymýšlí nové, protože tam ty bugy prostě mít nechtějí a protože tenhle druh kódu sami nepíší, neuvědomují si už, že ty nové postupy opruzují úplně stejně, jako ty staré. :-)
no it is the plural form of the text so this means the plural msgid
Fair enough. Taking it in stages is almost always better.
This is completely insane. Why in the world would we have multiple linkers?
Yeah.. Not 100%. It will break in the case where an object has a property called "pattern". Fixing it would be a significant amount of code.

Alternatively, I could just not check in this fix, in which case 'pattern' will just not work.
Issue filed: #1430
Missing second closing brace.

Also, not super important but: I don't know if we need to call this arg a "request" -- lots of our APIs refuse to do what you asked if there's a collision or other bad argument, so it might not be worth calling out this one case so much.  (And it's not like a "hint" type of API, where whether your request is granted or not is completely opaque and out of the caller's control).
Changing now. Once I resolve the conflict...
Instead of `onChange=`, we basically always prefer binding to the event using jQuery. So like,

```javascript
$(function () {
    $('[name="send_followup"]').change(function () {
        // the stuff you have in followup_changed
    });
});
You have other functions in this class that will raise reference errors if this returns None. Should this just fail hard?
you should be able to do Tab.STOPWATCH.ordinal()
And comma after the `]`.
What else could be there, other than span?
I think it makes more sense for `generatePublishUrl` and `generateIframeUrl` to accept a base-10 id, and convert it to the base-36 id. In many of the consumers of this API, you need to do the two-step conversion process otherwise
We could. Might just be extra noise though, since it's not really a requirement for butter itself to work as advertised.
also, we need to ensure that if someone interrupts a drag, the top function is called. we know it can be interrupted by right clicking during a drag.

If we miss the stop, the pointer-events attribute would be left on.
whitespace before closing bracket
Another `description` - if you felt like changing them.
Shouldn't sharedConnectionManager be set to null here?
Hashing without a salt is now less efficient. It used to be a constant time (free, really) operation and now it involves mixing using `combine`.
本家には報告しておいたので、  `(1- index)` ではなく `index` を使ったコードにしましょう。 `alpha` にあたる時には正しい結果が帰るわけですし。

https://github.com/davidhalter/jedi/issues/57
Our convention is to have this on the line above
I liked the fact that `create_user` and `create_superuser` already do a lot and we don't have to duplicate the code: https://code.djangoproject.com/browser/django/trunk/django/contrib/auth/models.py#L147

If we were to put those into model makers, I thought of something like this at first:

```python
def user(name=None, email=None, password=None, superuser=False):
    if not name:
        name = u'Karl'

    if not superuser:
        return User.objects.create_user(name, email, password)
    else:
        return User.objects.create_superuser(name, email, password)
```

However, `create_user` and `create_superuser` always save the created user.
good catch on the wording there
update this comment
oh good catch
It doesn't really matter. The leading backlash shouldn't be there though.
Move the un-minification logic into processMessages (within a helper method).  In the case that a developer wants to wire up their own onReceived handler we want to pass un-minified data to their handler as well.
Adding `that` doesn't gain you anything here.  There are no callbacks defined here that need to retain the scope, and `that` is more confusing to read than `this`.
To be done in #346.
写个sample的meaning例子吧。
Is libdl really a Linux-specific thing? I'd think if anything it'd be GNU-specific - so still needed on Debian GNU/kFreeBSD?
fixed (i will install code sniffer quite soon!)
`floor_validation_pattern` is required by mapserver to escape SQL injection possibility, I also don't remember what I will write, I will remove the comment.
You can get rid of this else easily
You're not reading the contents of this rowset.
Extra paren, not a big deal just nitpicking...

-[Unknown]
I would make this default to `''` (the empty string) - assuming a `files/` subdirectory by default will probably confuse people.
@takluyver do I need to be more careful about writing the .pyx files for Python 3?
Better to write this as two asserts so you know which test failed.
Got it. As of dc03369c07c6222c763cca8a80452608c8cce435, checkout creates empty dirs for every `GIT_OBJ_COMMIT` it finds as part of a checkout. That oughtta get you started; let me know if it isn't right.
This shows and hides the navigation on small viewports. Comment added.
Totally. I think across the codebase, we have every possible variation of ordering. :metal:
it's odd to pass data back when also sending an error - is anything expecting this?
I think this is another point in favor of my original true/false method. You're using numbers and numeric tests which is still bastardizing types just like using true/false (not you personally, but proponents of 0/1). While I claim true/false is much more readable and force you to remember the nuance that:

~~~ 
( exit 0 ) && echo true # true
~~~

Is one thing and

~~~ 
(( 0 )) && echo true # *crickets*
~~~

Is the complete opposite.
I checked on Windows and the menu mnemonics still work.  However, arrow keys don't seem to navigate the menus - did that ever work?
On the line below I bind repr to __str__ as well. This gives a consistent useful view of the exception whether you're using Ipython and get a repr or whether a traceback is raised and you get a str version.
This is quite a mouthful.  Maybe it can be simplified to something like this?
please use a better name
Naming could be improved for `$e`, perhaps `$stopwatchEvent`?
done, and exception doc added
$relative = true
$absolute = false
$this->context->getScheme() = 'http'
$requirements['_scheme'] = 'https' -> overrides $abolute and $scheme
With `$relative && '' === $scheme` later I can be sure it's really relative. Just checking $relative would be wrong now.
I'm not sure about this as in the RFC there's a mention for the special case where a user sends  -1 or 0 header values.
Have a look at my screenshot, it is *never* displayed (this is probably due to a media query side-effect).
@stof is ``auto`` cross-browser? As I can remember in IE < 8/Opera/Firefox3.x scrollbar are rendered even if not needed.
No. Replace usually takes an array, while here it would take a key and a value. I'll change it otherwise.
This is not equivalent to the current behavior. Currently, the YamlFileLoader allows you to use an array in a default value (even if it is a weird idea to do so) or in an option.
spurious println
I agree, although it's deprecated in Mocha:

```
  1) VoteCreatedJob#perform when the vote exists does not enqueue an ActivityCreatorJob for guests
     Failure/Error: ActivityCreatorJob.should_not have_received(:enqueue).
     Mocha::API::InvalidHaveReceived:
       should_not have_received(:enqueue) is invalid, please use should have_received(:enqueue).never
     # ./spec/jobs/vote_created_job_spec.rb:42:in `block (3 levels) in <top (required)>'
     # -e:1:in `<main>'
```
Lines 10 and 11 are indented more than 2 spaces.
这里被改回头了，文件路径错误
More yet-to-be-merged changes from maketx.cpp.