labofsahil/github-event-dataset-2012 huggingface dataset:
aapt: fixed junk path switch (it was not initialized)
update-tag-value-handler-26

Pick up changes from object browser version of tagValueHandler.js. Updated README.

Fixes #26
Openengsb 2338

Hi,

This is an example implementation of the build, test and deploy connectors with the OpenEngSBFilemodel. The path property has been removed, instead the path is passed to the build method. Note that the pull request is not against the HEAD of openengsb-connector-maven because I hope to stick with openengsb 2.x for my development work. If the change is OK conceptually I'll merge it with the recent build domain and maven connector changes.

The scm domain and git connector will be changed in a similar fashion.

Cheers,
Stefan
Fixed generating pdf name
Add CNA build and Set proper build name for AOSP
release notes updated according to known issues
CakePHP 2.x

This change are required to use the helper on CakePHP 2.x.

Thank you!
Tag lead content
Update Music/Files/res/values-ru/plurals.xml
Patch for create translation language dropdown

Copy values to keys in updateCMSFields' $alreadyTranslatedLocales since it's later used by LanguageDropdownField to exclude languages for which a translation already exists and LanguageDropdownField operates on array keys.
Also removed unused index/key variable $i from foreach loop that displays the existing translations.
Smooth delta robot calibration

Changed the deltarobot calibration so it works a lot smoother, and seemingly more accurate. Needs proper testing once we know the exact measurements of the robot.

Also added incremental motion functionality to the StepperMotor, it is only used in the calibration currently.
fix wifi on UHKI2 firmware
correctly build the Uri in parameters (fix #78)
Fix for bug that connects multiple profiles to users in some instances.

Here's the situation:

1. I log into Facebook as Alice
2. I log into the Django app using Facebook
3. I log out of Facebook
4. I log into Facebook as Bob
5. The app exposes a "connect with facebook" button, even though I'm logged in as Alice. I click on it.

The result: both Alice and Bob's Facebook profiles are linked with Alice's account in the app.

What this patch does is check to see if request.user does not match the user associated with the profile. If this is the case, log in as the user associated with the profile, rather than connect the profile with request.user.
No sabía quien es Shitan ni si en la ultima frase habla un chico o una c...

...hica, hay que comprobar eso.
In my case, the SQL Server could not join the contoso.com domain. I set ...

...the dns server to the IP Address of the domain controller and was then able to join the domain. I added this step as note to the HOL.
Looks great, Jason. Just the one substantive comment about assembling template filenames, and for the rest, forgive my pedantic comments.
ajajaj wtf
I repeatedly called read-byte on a decompressing-stream then called read-sequence to read the rest.
I now have underterministic behavior for refresh rate computation. I noticed that sometimes the video was good and sometimes the video was bad on the same source. After searching a bit I discovered that when I press the o key, the S element on the last line gives a refresh rate. When video is correct refresh rate is at 75 (that is the value given for the monitor by nvidia-setting on DFP0 item) and I have a good images and errors drop to 0. Unfortunately sometimes refresh rate is set to 50 and there is no way I can correct it.

I already have the initialization fix to 0.
note my rewrite in baded452887862
If you'll use `allSatisfy!(isCallable, T)`, this line won't be necessary.
clock.c for each cpu does clock_wait(ticks) and clock_delay(undefined time).
clock_wait_us and clock_wait_msec could logically go there but could not involve defines because the only clock.h is in /core/sys 
which is not appropriate for architectural defines.

rtimer-arch.h is defined for each cpu and  the core rtimer.h includes that.
So one way would be core/rtimer.h and .c
#define delay_usec             rtimer_arch_delay_usec
void delay_msec(uint16_t howlong) {while (howlong--)  rtimer_arch_delay_usec(1000);}

and /cpu/rtimer-arch-h
#include <util/delay_basic.h>
#define rtimer_arch_delay_usec( us ) _delay_loop_2(1+((unsigned long long)us*F_CPU)/4000000UL) )

gcc gives a warning if the expanded arg is > 16 bits:
191 delay_usec(42);
192 delay_usec(32768);
193 delay_msec(65535);
194 delay_msec(65536);
../../platform/avr-raven/./contiki-raven-main.c:192: warning: large integer implicitly truncated to unsigned type
../../platform/avr-raven/./contiki-raven-main.c:194: warning: large integer implicitly truncated to unsigned type

Some googling found a way to give a compile time error, but it's hardly worth the trouble:
#define CASSERT(predicate, file) _impl_CASSERT_LINE(predicate,__LINE__,file)
#define _impl_PASTE(a,b) a##b
#define _impl_CASSERT_LINE(predicate, line, file) typedef char _impl_PASTE(assertion_failed_##file##_,line)[2*!!(predicate)-1];
#include <util/delay_basic.h>
#define rtimer_delay_usec( us ) {CASSERT(((unsigned long long)us*F_CPU)/4000000UL<65535,usec_delay_too_long);{ 
_delay_loop_2(1+((unsigned long long)us*F_CPU)/4000000UL);}}
191 delay_usec(42);
192 delay_usec(32768);
193 delay_msec(65535);
194 delay_msec(65536);

../../platform/avr-raven/./contiki-raven-main.c:192: error: size of array 'assertion_failed_usec_delay_too_long_192' is negative
../../platform/avr-raven/./contiki-raven-main.c:192 warning: large integer implicitly truncated to unsigned type
../../platform/avr-raven/./contiki-raven-main.c:194: warning: large integer implicitly truncated to unsigned type

-----Original Message----- 
From: Nicholas Humfrey
Sent: Saturday, February 25, 2012 8:01 AM
To: David Kopf
Subject: Re: [contiki] Removed #define F_CPU from cpu/avr/dev/delay.h - it should be defined by the platform. (f6da78d)

No, I don't have a rt18019. I was just looking for generic delay routines. delay.c probably shouldn't be in the 'dev' directory.

I agree, it would be definitely be preferable to have define delay_us() and delay_ms() instead. It would be good to have an 
assertion that the delay period isn't too long for the CPU clock frequency.

--- 
Reply to this email directly or view it on GitHub:
https://github.com/njh/contiki/commit/f6da78d3238a082ab91f2e52caf0e2279cb555a7#commitcomment-1010756
Well, the remote routers need to handle the “nodes” setting, which the local ones do not. Hence there must be a translation between local and remote (unless the RemoteDeployer would not use local parsing at all, which is obviously even worse). If that translation is part of the router interface (so that the RemoteDeployer could do it generically) then local routers need to know about their remote counterparts, which is not possible due to cyclic dependencies between the packages.
This commit appears to have broken CI on Linux. The problem does not manifest on Mac OS X. See, for example, http://ci.jruby.org/job/spec-ci/jdk=sun-java-5,label=master,testtype=spec-ci_interpreted_19/1687/console for a thread dump.

I tagged the FIFO spec for now (in both 1.8 and 1.9 modes), so the CI tasks at least completes.
@jonleighton I think you're right.
@wycats I think you are right.
I have refactored it to use IWindowProvider.
Also grabbing works perfect on win32 with some extra OIS patches. :)
There are window resizing issues (cegui is not updated when window is resized).
Also the camera moves inverted for now, because in the past the relative movement was negative and it was not correct.

Also I don't have implemented Input::Config_CatchMouse, because I still don't understand it. When the user clicks into the window it still needs mouse grabbing and when leaves it still needs ungrabbing. So in what use case is CatchMouse usefull?
I think your issue is the same as the one I just fixed. Try adding it to your branch.
Thanks! pulling this out of the file function is something I have been meaning to do!
I see you removed the workaround for the repository named in lowercase letters. I think it would be nice to have the correct spelling. I was wondering before, why the repository is named falsely lowercase, too. Seems like you have something in mind with this. would you care to explain?
Any problem with this because I'm subclassing Rubygems? It doesn't use this yet, so it probably doesn't matter, but curious how that is handled.

Added, note that I'm on different branch now though too merge your with_constraints changes with mine. I'll merge to master soon.
@jravetch if you're having color issues, make sure you have the latest version of everything (git submodule update) and then run :PowerlineClearCache as well
I vote for a `class TaskButNotWrappedCallableProxyBeanFactory` class.
Not sure if you've seen, but the ADODB maintainer has responded to this change request at http://phplens.com/lens/lensforum/msgs.php?id=19013 -- might need to consider alternatives in case we dig ourselves into trouble WRT Debian packaging.
Comparison to undefined should always use ``` === ```
The space indent is four, not two blank spaces.
This is not actually going to provide any storage, because each cache object will cause a whole new Ephemeral handler to get created (and thus a whole new store). 

What I was thinking here is that the injection of the Ephemeral cache should be moved into the CacheFactory style objects- the Cache service in the Symfony bundle, for instance, or the Box and Manager example classes we have, should detect that no handler is present and then create and store an Ephemeral instance. It's this reuse that is really key.

Continuing that line of thought, it may also make sense to have these frameworks inject the memory handler before *every* handler (using the multihandler)- the in script store has huge performance benefits, and if the frameworks simply skip that step they'll be missing out.
Moet nog een <br> achter of je moet het in een <table> stoppen...maar nu komt alles achter elkaar
I think all platforms support JPEG (GDI with HAVE_IMGDECMP_DLL) and Android also supports PNG.
Just put in a pull request to fix this: https://github.com/ajaxorg/cloud9/pull/840
I don't think this is the right solution here - one needs to be able to use the fog provider directly rather than through a subclass like zerigo does (only really there for compatibilities sake)

The zerigo class should just munge the credentials hash before passing it to super for creation of the @client
Hah, I will assign more Redmine tasks, seems you have a lot of free time :)
Is it just me or are the LDFLAGS values all the same?
Может таким обрахом сохраняется возможность исправить бонусы через таблицу в базе, даже если данные есть в dbc?
The first problem: Stun is applied with 0,5s delay - bug.
This looks suspicious -- list with a null element?
PHP may be a dubious authority, but Perl and Ruby are legitimate and they both call these operations `shift` and `unshift`. Python [doesn't have methods for adding or removing an item at the front of an array](http://code.activestate.com/lists/python-list/323234/). Java of course [makes up verbose, awful and unmemorable names](http://docs.oracle.com/javase/7/docs/api/java/util/Deque.html) for these operations. JavaScript uses [shift](http://www.w3schools.com/jsref/jsref_shift.asp) and [unshift](http://www.w3schools.com/jsref/jsref_unshift.asp). ActionScript also uses these names. So it seems that we should just go with shift and unshift.
Confirmed. This commit freezes server on Windows when character logins to world
@Undergarun Sí, pero en los creditos de los archivos pone "Author: rsa"
Would this be better?

if(this.dialogSpecificClass){
	var c = require(this.dialogSpecificClass); 
	t.dialogSpecificWidget = new c({dialogSpecificButtonsSpan:t.dialogSpecificButtonsSpan}, t.dialogSpecificOptionsDiv);
}
Can't we just pass outerContext down from exec*() all the way to predTransition() where we need to do the test?
The comments at the head of your blocks are great, but if I may suggest that you do away with the line comments at the ends of your lines that are describing what the line does. For example:
`$this->lineString = trim($this->lineString);      //Trim any whitespace from the file line string.`
This line reads redundantly since there is nothing unique going on in here. You might consider adding a line comment for something more complex or give a reason why you were adding the particular line rather than explaining what it does, but much of Reader.php would actually read *easier* without the the multitude of comments.
So, my final answer isn't coming out right. Can you guys take a look at this?
Does *Visual Studio* correctly recognizes C vs. C++ on these extensions, `cc` and `C`, without explicitly setting the compiler?
See issue #16
Personal recommendation is you make an online builder or make a command line npm builder so 
 - `npm install augment` 
 - `augment build map filter`
Cool, I will update this later this week. Any thoughts on the python install problem though? (mentioned in original issue)
Why does it matter if some browsers support fractional values? We're just comparing to the center of the element.
this is a bit bright. you should use `@error-red`.
Should be "customType".
Tried to create a collatex-demo pipeline. It failed, but installation generated all these changes. Not sure if this creates problems!
Sorry, could not figure out how to create new branch and save to branch.
Here's another one, also merged:

http://bible.fhl.net/daily/read.html
it could, but i like the way it looks, required, then optional. The total number of lines stays the same, clarity improved imho.
I'm super stoked to have justin bieber in the codebase, (and glad I didn't have to be the one to introduce him)

do you know why this would be failing on CI?
http://travis-ci.org/#!/diasporg/diaspora/jobs/520758
I ended up writing my own format so that I could mark individual files that were important.  Since some packages are only one file, it doesn't make sense to create a *tar* of whole repo every time.

It would be nice to just see a snippet if that's possible. you could email it to me or something?
Yeah absolutely am.

I just spent half a day wrapping the sentiment analyser in http/json rpc stuff, and realised I was on crack.  All the edge cases you need to handle with http (like retries, blocking until the server is back, queueing) are all inside zmqrpc.

Pull request?

g.


On 8 Jan 2012, at 20:31, Alex Michael wrote:

> Cool. I am actually hacking a bit on zmqrpc - mostly for learning purposes but I think I got it a bit cleaner and more tested. Let me know if you are interested in that.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/geoffwatts/zmqrpc/commit/7674d929df38af79e95e9bf98fd6c111e5f86626#commitcomment-847965
Go for it - it'll be much easier than merging conflicts :-)
Did you see that this breaks the build on Travis? 

http://travis-ci.org/#!/rubinius/rubinius/builds/696768
Here a return is also missing. Same at line 527.
Try to change some code,anywhere as you wish~
Have you considered sending a pull request? It'd be good to have this in the gem.
Nice, this looks promising!
What are we trying to say here?  That you can't have XA and batch-mode?  Why not?  Agreed it may not be the default, but still.

Also, this check could be made in the TransactionConfigurationBuilder.validate() and BatchingConfigurationBuilder.validate() since you can then change the config if needed over there.
Not sure we should have a default.
Module without outputs? Which one required this change?
fuck is dis
Really gotta re-read that git documentation. ._.
I see. Then I think that should be fixed in p.a.linkintegrity's patch rather than here, since merely skipping the teardown like this could can lead to a *really* hard to debug problem (tests using a cached db connection from a previously set up test layer)
No this devel md5 hash shouldn't be there. Pushed 32674f3 reverting this line. Thanks
This function name is going to be completely impenetrable within a week. Something that's a little less in-the-moment, please?
This isn't working to resolve my _update.html url
I couldn't really see an argument either way (since I'm not trying for browser compatibility). I guess I should run some jsperf tests to help decide.
Радует... еще после релиза fdb admin'а (может, уже пора переименовать в db_admin?) от Лемура с поддержкой мускула, его тогда просили сделать возможность делать экспорт/импорт БД, исполнение SQL запроса
Ideally, should we not keep our design within the stylesheets and instead define

#task_sidebar {
   @extend .pull-left
...
}

That way, we don't start defining the design within the html.

What do you think?
Hey Buddddddy :P
Why all of this commented out code?
That's what i mean by personal preference. I'm not referring to you, Paul, i'm referring to the Zend project. The Zend project decided that they should use Suffixes and Prefixes. It is project-centric and personal preference. PSR-1 should not list rules that were decided in standards like Zend, PEAR or Horde. It should list generic rules that can be adopted by major projects and allow them to extend and implement their own, such as, suffixes and prefixes.
This commit breaks grim, which exits with:
  Could not find colormap yr1_pal2.c!
Happened on bi.set, and probably on a palette just because it's the first section.
Y con este te acabas de fumar mas cosas que con el otro... Revierte este anda...
What is the benefits of this dependency ?
Als ge eerst update feeds doe, en dan pas copy config gaat bv de OMF feed niet geupdate zijn, of geinstalleerd...
I think you need to Forgot Password link
I'll edit it after its been merged, don't worry about that. ;)
Why don't use sendfile for linux too then?
All autogenerated files should definitely be excluded.  But I doubt that this file is autogenerated - it is present in the original UHD repo.
Fuzzy translations are not used. You need to remove the line after correcting the translation.
I tried out a couple of scenarios and I think it's worth keeping NonNullableTransientDependencies. It really helps to have all the details for logging.
No, because we have to consider mobile devices too and that would take up too much vertical space on smaller screens.
John, the documentation looks good. Thanks for writing it. In the future can you use the (@param x) (@returns x) syntax so the documentation meshes across the code base?
You'll want to change railsready_recipes_path to $railsready_recipes_path to access the variable.
Again put images through JS from the plugin.
Then if we have no problems with cleaning up names and what not, can we refactor most of the code to implement an actually consistent casing convention?
You can't blame anyone actually. 
The motto is "Help TC help yourself."
but it can be "Blame TC blame yourself" as well.
what about `0O[0-7]` (uppercase) etc?
Проверь пжа, работает ли эта ссылка. Мне кажется edit_album_path необходим параметр (id)
Just a minor update. I think onCreate() is too early for making any UI job. If I look through ApiDemos I can see they do the UI job in onActivityCreated() whenever they use ListFragment. This way they are sure that activity is actually created by the moment they call setListAdapter(). If they extend Fragment they do UI things in onCreateView().
所以可以保证一定会 on end ?
ahh, ok. das hatte ich bersehen. :)

2012/1/2 DracoBlue <
reply@reply.github.com
>

> This is already the model. SInce selected_field is actually a backbone
> model of the griditem.
>
> See
> https://github.com/DracoBlue/sudoku-android/blob/master/dummy/static/js/PlayingFieldGridItemView.js#L33where the event is fired and in
> https://github.com/DracoBlue/sudoku-android/blob/master/dummy/static/js/ChooseColorOverlay.js#L28it is fetched.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/DracoBlue/sudoku-android/commit/c4b5ca915e160a0e28d842fd5e9608af5bd53060#commitcomment-831941
>
Yeah, I don't like this either. So we should set the default value to n_samples? Is this a good thing?
It will, I guess I hadn't seen the pSample being NULL as CComPtr<IMediaSample> that is used for the IMediaSample** source itself is always in the stack.

it should be "if (pSample && *pSample)" to make sure that NULL pointer is not referenced.
This is stupid and doesn't work right all the time.  You should have used cURL requests...not file_get_contents FFS
Can you undo the doc changes. I want to use dox and plates
Btw, the commit 451a5b4fc216ac40eb7507cab5fe06436ea4e00 you linked above is not a *previous* one as it has been done 3 months **after** this one, and it was removing another class (different namespace) added only in its parent commit e83a3344e9f42d0f0ad8cec6e135cf8f7eee21b so it was probably an experiment
On 08-02-12 17:20, Bruno Binet wrote:
> Do you mind using collective.eggproxy repository from my organisation rather than this one?
> https://github.com/camptocamp/collective.eggproxy
> We plan to add a collective.eggproxy team and add you to that team.

Actually, I'm not a member yet, so if you want to do something *now*, 
perhaps it is handier if you copy my latest commits over to the new 
location? I'm going home for dinner now :-)


Reinout

-- 
Reinout van Rees                         reinout@vanrees.org
                http://reinout.vanrees.org/
"Military engineers build missiles. Civil engineers build targets."
I was not sure if I could delete the AndroidAnnotationsWebsite folder at the time, now it's done ;)
should call this->Modified() when the string is different

Why not handling strings the same way all the strings in MRML are handled ? (using const char * and vtkSetStringMacro). It's not ideal but at least it's consistent.
This is a good tweak, but perhaps we should have some kind of auto-completer field here?
Si, los fantasmas persiguen siempre al mismo target.
This won't detect if the section is commented out.  Use token_get_all and iterate through the source to find specific parser tokens which match what you need.

http://us3.php.net/manual/en/function.token-get-all.php
$sehir değişkeni bashrc de  belirtilmiştir. Deneme amacıyla kurulumda scripti çalıştırmadan önce export sehir=Sehriniz komutunu girmelisiniz.
It is to be able to use `loop.cycle()` within the inner loop, without this the `tr`'s class would be 'flDebugOdd' for each one (don't ask me why, but it works, i had this tip from the #pocoo channel ^^)
I kept thinking that, you're right though
The stop button doesn't work yet.  I have to look into how to kill a running service...
@vladislavbyk А моей жене наоборот кажется :)
disabled test detected
wlan_mac and wlan_tool needed by baddr_read, ;)
but thats no problem at all, but if you have problem with wifi on next build, just at this too

because we will merger some patch for wifi option and have FC if this not applied, just in case, Lol :D
Duplicate class definition
de que trata este deprecated?
I signed the other projects because dbup was already signed. More than happy to remove this.
Some fairly quick testing shows that it's executed once per classloader as intended.
import numpy
a = numpy.arange(10)
hash(a)
works for me
I think we can. I don't think there is a problem with par_loop having a lot of documentation.
Sorry, that doesn't work, but this does:

a = numpy.arange(10)
a.__hash__()

aha, but that's not a hash of the contents.
This change makes an already large function quite a bit larger.  Can we extract the job/service building into it's own function?  It's also identical behaviour.  This could probably be a single function which takes the configs and a attribute name, and returns the full list.
This is solved in https://github.com/akka/akka/pull/675
which is not merged to master yet.
Probably that will go in first and then you will adjust. I will guide you to what changes are needed when it's time to merge with that.


/Patrik

8 sep 2012 kl. 01:26 skrev Helena Edelson <notifications@github.com>:

> In akka-cluster/src/main/scala/akka/cluster/Cluster.scala:
> 
> > @@ -149,7 +149,9 @@ class Cluster(val system: ExtendedActorSystem, val failureDetector: FailureDetec
> >    /**
> >     * INTERNAL API
> >     */
> > -  private[cluster] val clusterCore: ActorRef = {
> > +  private[cluster] lazy val clusterCore: ActorRef = {
> He had sent me that and I pulled it in. In Master it is different https://github.com/akka/akka/blob/master/akka-cluster/src/main/scala/akka/cluster/Cluster.scala But I know after the pull the NPE's went away. I'll look into it.
> 
> —
> Reply to this email directly or view it on GitHub.
>
this should be `system.systemActorOf(props, name)`
This test checks if it can successfully deploy actors to a remote system through NAT and then receive a message back.  However it does not cover all the angles you mention above.
ah, I thought it was your old self...
Hakat upp dig?
include the code from SupervisorAskSpec here (once moved out)
plus: there’s a superfluous dot on this line
singleton isn't very self describing when imported like this, perhaps rename it to `singletonCollection`
tried it, but it only makes the scala api more awkward, since the tuple construction feels wasteful. I'd rather have a nice scala api, since that is what we use in the router like this:

    override def capacity(nodeMetrics: Set[NodeMetrics]): Map[Address, Double] = {
      nodeMetrics.collect {
        case HeapMemory(address, _, used, committed, max) ⇒
wow. yes please. Some tests for this sucker could definitely be added as well.
ObjectContent uses this.  The scan generates an array of arrays with the match groups.  The first group is the text between the chunks, then the groups are groups per chunk type.  We find the index of the first non-nil and get the class and range it lands in.  Use the range and class to extract the parameters for the class and create the new chunk.

Not sure if flatten and compact are overkill here.  The pre_chunk bits might be nil, might be blank? too, maybe filter for that ...
I pulled the refresh stuff into a pull request to develop.  Doesn't get at the deeper issue, but it is a start.
It may be that actually entirely separate package_search functions are wanted.  To be honest the way we get the packages one at a time and process then feels quite nasty and recursive but refactoring it feels outside of the scope of what is wanted here.  It is one thing to add the ability to do something and something else to improve the efficiency and they should not be combined unless there is very good reason as the likely hood of breaking things is high.

Also the needs must be well understood.

is it that we are not wanting context['for_view'] but context['for_dashboard'] or some other way of distinguishing ie context['view_type'] = detailed|quick or something like that
extra space before "distro_"
True. This was according to how I had visualized implementation initially. Removed default values.
Given that the deleted flag is so integral to bindings, it would be good in the docstring here to indicate whether or not deleted bindings will be returned.
[keep lines under 100 characters](http://siq.github.com/front-end-development-guidelines/#lines-should-be-100-characters).  we tend to break things up by property access:

    $sortColHeader
        .removeClass('sort-null')
        .addClass(col.order === 'asc'? 'sort-asc': 'sort-desc');
cool!
for now should we have it fail loudly if there's no trait arg?
Test? :) getTimeline is full tested, this one should be too
@cptspiff has the final word these days on PRs being merged ;-)
optionparse is no longer a football to be passed around.
+ this.getClass() = " and " + t.getClass()
Add the license header to the new files
It is a private method.  But yes, more doc wouldn't hurt.  documenting private members is still a thorn in my side.  I guess I should stop complaining.
alternative implementation in d35e3ca22ef4313b6d8a3d169355f469ffe672f6
Just for the record, the correct English word is "index" here (the plural being "indices" or "indexes").
……写个函数到 libfcitx 里面吧…… ime.h 里面……
This should be read from `NSImage.name`. There's no need to set it separately here.
Two private categories on the same class shouldn't declare the same method – it's duplicative and just asking for breakage. There should be a header file (e.g., `TUITextRenderer+Private.h`) that exposes this instead.
As mentioned in the main discussion, this should again be a cast to `id`.
Having to make an explicit call to `-configure*` does not seem like an elegant solution. A custom initializer with an enum of switch types would be better.
This entire thing is also in spaces
Maybe turn this into a Makefile? I'll write one for you if you want.
Long lines + style errors.
Prefer `reinterpret_cast<uv_handle_t*>(&handle_)`
One minor suggestion: use null instead of "" here, to avoid object allocation when we deserialize it. (!)
This metadata you may take it from the interface.Or am I wrong?
What about $asFirst ? :))
The -> the as suggested by Sun's Javadoc convention.
the default on this should be False to maintain previous behavior
`traits.Bool(False, usedefault=True, ...`
missing a '()'
а может совместить и таймер и счётчик? то есть счётчик если сработал то обнуляет таймер и наоборот?
не переведено
Please can you add a code example? It's a bit unclear what the functionality does at the moment.
Put in the same indentation of `and`
because it is the same during the main cycle, so it could be cached
Is there a reason behind this change?
The modifyTypes are a clear win for clearity at least.

The backend is not the undiscovered genius we might have hoped. The finalizer gets duplicated. I don't think it bears sweating (it's only one case in one transformer) but I can pull it if you want.
```
@@ -88,14 +88,21 @@ public class T extends scala.reflect.api.Trees$Transformer {
       : astore
       : aload
       : invokespecial #                  // Method scala/reflect/api/Trees$Transformer.transform:(Lscala/reflect/api/Trees$TreeApi;)Lscala/reflect/api/Trees$TreeApi;
-      : astore
       : aload
       : invokevirtual #                  // Method bippy:(Lscala/reflect/api/Trees$TreeApi;)Lscala/reflect/api/Trees$TreeApi;
       : pop
-      : aload
       : astore
       : aload
       : areturn
+      : astore
+      : aload
+      : invokevirtual #                  // Method bippy:(Lscala/reflect/api/Trees$TreeApi;)Lscala/reflect/api/Trees$TreeApi;
+      : pop
+      : aload
+      : athrow
```
@xeno-by I mean, if you see a ValDef individually, which came from a self-type, can you detect that it did? Or do you have to have this knowledge from the context? Before there was a question from you "what is set here". Assuming you mean modifiers, I checked, the only one set on the ValDef is PRIVATE. Does this info help us somehow?
Okay, then call sys.error.
To be honest I like the current version better. It's more verbose, but it puts an accent at the main event in `macroExpand` - the `!wasDelayed && !nowDelayed` branch. In the proposed variant it will be buried under one more if.
Here I meant "with the previous implementation".
s/yet/still/ ?
Here's a pertinent piece of code:

```

    @property
    def is_comparable(self):
        is_real = self.is_real
        if is_real is False:
            return False
        is_number = self.is_number
        if is_number is False:
            return False
        if is_real and is_number:
            return True
        n, i = self.evalf(2).as_real_imag()
        if not i.is_Number or not n.is_Number:
            return False
        if i:
            if i._prec != 1:
                return False
        return n._prec != 1
```

A problem with this, however, is that for `q=(-1)**I` q.is_real comes back as False. That's a bug.  Otherwise, the idea from the `n, i = ` part that tries to be careful about reaching a conclusion. If the precision is 1 then you don't know what the number is and in the assumption system, you should return None in that case, not True or False as is done in the above.
According to me, if the expression has been simplified correctly to smallest SOP/POS, then we dont require a fifth parameter in the fingerprint. Can you give an example where the fifth parameter is required?
There are the functions is_odd, is_even which do basically the same thing (determine the signature of a permutation)
We usually differ from PEP 8 in SymPy for `*` and `**` because it makes reading expressions like `3*x**2 + 2*x**4` easier.  Here I would say it's justified, though really either way looks fine.
is it necessary to write this as MutableMatrix since Matrix is a synonym for it. (And I move to deprecate MutableMatrix in PR #1535.)
Is there a method to ask if an `Equality` (or more generally a `Relational`) is satisfied always (return `True`), never (return `False`) or sometimes (return `None`)?
Sorry, hit "Comment" too soon.  I mean that `remove_zeros` only removes one zero, while applying `remove_zeros` using `exhaust` will remove _all_ zeros, that is, the operation of applying a rule until there are no more changes is generalised in `exhaust`.
Do you think that mentioning this function in the docstring of the actual `canonicalize` will be of interest?
There is one further modification -- the use of order -- that is in
#1462, too. barring errors, I think I am done with that now. But you
and others may have comments.
I suppose this should be `_groupby`.
You shouldn't forget what you should be coding :)

The same applies for one-liner if statements; ones without the curly brackets.

We shouldn't try to do it fool-proof.
# missing.
Just wondering what would happen if BOUNDARY_CONDITION_PRESSURE is applied to a Linear Elasticity/fluid equation set since there may not be code to handle this and/or no element residual etc (is it taken care of elsewhere?)
Testify is Python 2.5-compatible; if you're going to use `with` you need to `from __future__ import with_statement`.
I would just dump the temp var: max_elements = len(iterable).
Short docstring.
Replace tab with space here. Same for the following line (line 81).
Сделать обёртку над weakref.proxy, проксирующую repr
Could this just be `parent.touch`? Won't each parent handle `after_touch` itself?
If stopActor were to be called between startActor and findActor, then findActor would throw an Exception.  Does that only happen when the escalation rule is removed? If so, this seems sensible - not really a race condition, just potential for an exception in findActor.
these are not options to geoNear (double checked the wiki). So removed.
No, they should be kept for IDEs
Ist gefixt.
You mean remove the declaration of current_object and just create it in every each?
+1, censor_text should be run before truncate. I'm not sure what you're saying.
{} on own lines
These probably belong to `phpbb_cache_driver_sql_generic` instead.
just call serialize on the actual structure here, don't hardcode the serialization format
Should be converted into a proper docblock.
Oh, I just realised avatar_width is also part of the URL. In that case maybe add a function for building the URL and just make `https://secure.gravatar.com/avatar/` a constant.
I think logic for dealing with the type from plugins belongs in plugin-specific code and shouldn't leak out to here.
i do a process.exit()
这个还是要保留，开发环境下有些生成器要生成 factory_girl 的测试数据
I the '1' worth giving a name to?
Can you add a block of comment explaining the details about this routine, and why we need to do the two cycles to pre-compute the length?
More generally, I'm not sure that Forms are the best way to display this information... It would probably be a more readable Confirmation Screen if it displays text rather than readOnly forms. What do you think?
if this is an openmrs Patient, prefer not to name it *raxacore*Patient. Else it appears to be something different. just "patient"?
HeightInFeet
text is wrong. 
* not a drug purchase order - this is a BillItemAdjustment
* also, this is the ```onShutdown``` method, not the startup method
There was a function ```old_style()``` that in addition to some ```TEST_COMPAT``` code also removed the ```surface_altitude``` coord and the cube's aux_factory.
don't you have to cap the n_threads to device limits here?
I am not that good in english, but `To and Fro` is a common expression.
It overrides any future unexpected behavior and side effects. I sleep better knowing I mocked out as much as possible.
Why save it two times?
this check will not work properly anymore as ``$value`` will be encoded
Partial teletext line support.
Without this, you get errors when dealing with numpy.arrays. Switching the operands around allows for a proper short circuit.

Here's an example:
<pre>
test_initialize_realizations (tests.calculators.hazard.classical.core_next_test.ClassicalHazardCalculatorPreExecuteTestCase) ... ERROR

======================================================================
ERROR: test_initialize_realizations (tests.calculators.hazard.classical.core_next_test.ClassicalHazardCalculatorPreExecuteTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/larsbutler/proj/oq-engine/tests/calculators/hazard/classical/core_next_test.py", line 127, in test_initialize_realizations
    self.calc.initialize_realizations()
  File "/Users/larsbutler/.virtualenvs/openquake/lib/python2.7/site-packages/django/db/transaction.py", line 217, in inner
    res = func(*args, **kwargs)
  File "/Users/larsbutler/proj/oq-engine/openquake/calculators/hazard/classical/core_next.py", line 188, in initialize_realizations
    self.initialize_hazard_curve_progress(lt_rlz)
  File "/Users/larsbutler/proj/oq-engine/openquake/calculators/hazard/classical/core_next.py", line 222, in initialize_hazard_curve_progress
    hc_prog.result_matrix = numpy.zeros((num_points, len(imls)))
  File "/Users/larsbutler/.virtualenvs/openquake/lib/python2.7/site-packages/django/db/models/fields/subclassing.py", line 99, in __set__
    obj.__dict__[self.field.name] = self.field.to_python(value)
  File "/Users/larsbutler/proj/oq-engine/openquake/db/fields.py", line 160, in to_python
    if value and isinstance(value, (buffer, str, bytearray)):
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()
</pre>
If vp is not `vpStack`, then it does not have `children` slot.
Setting one field of a union then accessing another is undefined behavior. `(void*)(uintptr_t)usage` will quell the warning about casting from function pointer to `void*`.
I actually thought about having all of them call a single function as default earlier today, but I like your first suggestion a lot better.
Why render_to_response instead of the render shortcut?
I came across a bunch during superenv work, but I fixed most of them (with superenv), however there are some, often because the various things are built with CLT and then CLT is removed, and this leaves them broken, like python-config.
Yes, I'd like that. Maybe `macosxsdk 10.5` looks a bit better (you're calling `v.to_s anyway`).
Is this step needed for the stable release or just HEAD?
Pointless test, just omit it.
It does look a bit silly and really serves no purpose. I'ĺl remove it.
Could you do this in a functional style?
```
@bright_stars = grep { defined $c->{"TYPE$_"} 
                        and $c->{"TYPE$_"} =~ /BOT|GUI/
                        and $c->{"GS_MAG$_"} < $min_mag } (0 .. 16);

if (@bright_stars < $min_n_stars) { ...
```
I hate to ask this, but can you use an array here rather than a List? Everytime you for loop over a List that allocates an object, and our garbage collector isn't that awesome on Android.
I think I'd prefer just using `this._dirtyPaths[path]` without comparing to `undefined`--seems more idiomatic, and we know we won't be getting into any weird cases like comparing against 0, "", etc.
In the "input" case, we're actually committing the newColor with some correction to the user input (eg. ``rgb(0,0,0)`` -> ``rgb(0, 0, 0)``. Here we're restoring the previous valid color since focus is leaving the text field and we can't use the user input color if it is invalid. So the "if" block above is restoring the previous valid color to newColor and we call commitColor with newColor here.
This is just encoding existing file and folder names retrieved from disk. So, there won't be any '?' or ';' chars. This is not user input.

Maybe I do not understand your concern.
OK, I'm removing the line above the switch.
We can remove this commented-out stuff, right?
Possibly, yes. Once this pull merges I can file a code cleanup issue.
What do you mean by every exit? `window.onunload` or something like that?
You might be right actually. I couldn't really try it online because I'm pretty much blocked by https://github.com/ajaxorg/cloud9infra/issues/3068. In that case, it may be that the relative path c9.io gives us is different. (Which also sucks.)
or "server"?
are these two actually used anywhere?
"localhost:4567" is actually "hostAndPort" than "hostName", might mislead clients if they don't see this test example.
We just don't have a text yet for that, and I'd like to make sure we notice it when looking at the front-end
If you use only one parameter, why do you have two ?
You didn't, but it is in the main repository. :-) I am saying that you should check the diff of the pull request and you will see the changes you are doing. And there you can see that you are throwing out many useful classes.
Please format consistently. Use spaces before `{`. Spaces after `:`. Indent properly.
Corrected, but I don't know how to get query string from previous url..?
Smart! I'll take it.
spacing beside the right bracket
Whiiiiiiiiiitespace
I don't think we set anything else on the options object but you're right in that I probably shouldn't ignore the entire object. I'll update now.
And these 2
I was incorrect, this and the one below it should be max
Ehm, can we get spaces around `=` here please? :)
Ich weiß, dass zu header('Location.. eigentlich eine komplette URL gehört. Da wir aber in REDAXO sowieso an einigen Stellen per relative Pfade weiterleiten (z. B. Zeile 284 hier drüber), und es scheinbar keine Probleme macht, würde ich das glaube ich hier auch machen. Denn gab es nicht auch irgendwelche Sicherheitsprobleme mit $_SERVER['HTTP_HOST']?
Do we want to make this configurable so that the vendor name depends on whose extension it is? Or do you think this belongs under Salsita always as the platform vendor?
RedHat is actually 1 word with the H capitalized fwiw.
Please keep indents (and other things) PEP8 style. :)
在这里可以举个例子。
Kannattaa miettiä yksilöllinen toiminta getille ja postille. Esim. Get näyttää lomakkeen ja Post käsittelee lähetetyn lomakkeen.
A comment here describing what the job does. Also add Apache header.
It looks like a bunch of the changes are re-indenting things. Can you fix this to follow the existing indentation? It makes it easier to follow in the template code.
I check if `chars` contains something because in case that there are more than one character or no one of the speakers starts with the given character we fall back to `chars[0]`. I wrote a comment after the else clause, shoulf I put it befor the `if`?
It's probably a `virtualenv`. I'm using `.git/info/exclude` for local adjustments because people may use different names for these, I for one use `env` without a dot.
That wouldn't work. In `Encoding#to_utf8`, it uses `@mapping[c] || c` so that characters which are not mapped to a different character are passed through unchanged. To use `Hash.new(PDF::Reader::Encoding::UNKNOWN_CHAR)`, all valid characters would have to be in `@mapping` (and mapped to themselves). Perhaps that wouldn't be a bad idea, but it would be different from the existing behavior (which this patch emulates, just with better performance).
Don't add Readme to the clean paths.
could you check the reworked version ?
false, false, I have no idea what that does.
"In order to" ?
Storage -> this
e965315c68a2845ae3676062274751cc7ac3e1db
not sure what you mean by "live"
{ok, SeanCribbs},  %lololol
Cleaned it up a good bit, pretty close to what you suggested.

*Nikhil Goel* | Vanderbilt University, 2013 | Computer Engineering,
Economics**

LinkedIn <http://www.linkedin.com/in/ngoel1> |* ngoel36@gmail.com | ( (256)
503-9183 | NikhilGoel.me <http://nikhilgoel.me/>



On Fri, Nov 30, 2012 at 6:14 PM, Luke Steensen <notifications@github.com>wrote:

> In app/views/videos/show.html.erb:
>
> > @@ -19,7 +19,19 @@
> >      </div>
> >      <div class="video"><%= image_tag (@video.thumbnail_url), :height => 320 %></div>
> >      <div class="stats">
> > -      <span><strong><%= pluralize(@video.views.count, 'View') %></strong></span>
> > +      <span>
> > +        <strong><%= pluralize(@video.views.count, 'View') %></strong>,
> > +        <strong><%= pluralize(@video.bookmarks.count, 'Bookmark') %></strong>
> > +        <% if !current_user.bookmarked?(@video) %>
> > +          <%= link_to "Bookmark",
> > +            listings_path(list_id: current_user.bookmark_list.id, video_id: @video.id),
> > +            method: :post %>
> > +        <% else %>
> > +          <%= link_to "Remove Bookmark",
> > +            listing_path(current_user.bookmark_listing(@video)),
> > +            method: :delete %>
> > +        <% end %>
>
> It'd be easier to read if you inverted this conditional.
>
> if bookmarked?
>   ...else
>   ...end
>
> Or if you really like it this way, use unless.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bitcast/bitcast/pull/22/files#r2284434>.
>
>
Looks like you indented with spaces instead of tabs here.
Whitespace should not be indented
this is still the old behaviour for now, #28 proposes how to fix this.
In Python 3, rpy2 writes str (i.e. unicode) to our function, so trying to decode it again will fail. I think `IPython.utils.py3compat.str_to_unicode(s, 'utf-8')` should do what you want.
`nargs=1` is default behavior for optional argument

```python
In [4]:
import argparse
parser = argparse.ArgumentParser()
parser.add_argument('--filename')
parser.add_argument('positional')
vars(parser.parse_args(['--filename', 'a', 'b']))
Out [4]:
{'filename': 'a', 'positional': 'b'}
````
In terms of naming, is `flags` okay? Or should we make it `compiler_flags` ?
On Tue, May 8, 2012 at 2:07 PM, Jason Grout
<reply@reply.github.com>
wrote:
>> @@ -46,46 +49,45 @@ var IPython = (function (IPython) {
>>          return msg;
>>      };
>>
>> -    Kernel.prototype.start = function (notebook_id, callback) {
>> +    Kernel.prototype.start = function (notebook_id) {
>
> If I understand things correctly, decoupling the notebook and kernel is effectively what we did with the Sage Cell server (any webpage can embed a single cell and execute code as part of that single cell, and in fact, any webpage can just call the aleph.sagemath.org/service/ URL to do arbitrary computations without even embedding any cells into the webpage).  It would be *great* if IPython had that sort of thing.

With the recent pull request, I think anyone can simply grab our
kernel.js file (and a few other basic JS dependencies) and start to
talk to a kernel on an IPython notebook server.

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/1697/files#r791411



-- 
Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com
It's probably not important. I guess the main time it will come up is a typo in the path, which the user should notice easily enough.
It works fine for me in Linux.  But I suppose I should duplicate a similar methodology as the code above:

```python
pyx_file = os.path.join(lib_dir, module_name + '.pyx')
pyx_file = py3compat.cast_bytes_py2(pyx_file, encoding=sys.getfilesystemencoding())
with io.open(pyx_file, 'w', encoding='utf-8') as f:
    f.write(code)
 ```

Any idea why the `cast_bytes_py2` line is there?  That seems totally crazy to me...
Real bug I was not expecting RDATE/EXDATE to be multivalued. Now we correctly handle both multiple properties and multiple values per property.
Doesn't this break any existing code?
Should we add a length check here for `git_reference_name`'s result to avoid running off the end of the buffer if it doesn't start with `/refs/heads/` for some reason?
No problem keeping the standard site_js block. It adds a bit of JS we don't need, but it's pretty minimal.