labofsahil/github-event-dataset-2012 huggingface dataset:
Yes, we can arrange that. In fact we already install it on a temporary location first:

```
topdir = tempfile.mkdtemp()
...
install_root = os.path.join(topdir, 'install_root')
```

And later copy the grubby binary only to `/sbin`. I avoided using this `install_root` as the definitive place for  the installed grubby binary because it would require either a known (and possibly new) location on the filesystem, or some way to keep this location known between boottool runs.

Notice that the tool takes care of saving the current grubby binary as a backup first. So the impact on system should be fairly minimal.

Still, this is an RFC, so I'm open to suggestions!
Shouldn't the last commented line be
```#int-type=str```
? It works like this and I didn't try without the equal sign.
enhhh we should maybe not broadcast his true identity?
thanks :)

On Thu, Mar 8, 2012 at 2:36 PM, Sean Caetano Martin <
reply@reply.github.com
> wrote:

> I actually didn't submit it to npm. I will asap.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/xonecas/disqus-node/commit/d656bdd0320816fa8f59e64702c8719a66fa1cd2#commitcomment-1062612
>



-- 
*Joe Levy*
Duke University, Computer Science
919.886.6563
jal50@duke.edu
``` css
@mixin icon($row: 1, $col: 1)
{
	position: relative;
	&:before {
		position: absolute;
		left: 10px;
		top: 7px;
		display: block;
		content: '';
		@include sprite('../images/icon-sprites.png', $row, $col, 16px, 16px);		
	}
}

@include icon(1, 3);
Having to use MacRuby, especially when using regular Ruby for other things, was a really unnecessary barrier to entry. This makes it possible for people to use CocoaPods out of the box with Lion.
Is railties ~> 3.0 really correct?
I'm using railties ~> 3.2 in my project and therefore can't use dm-rails..

Maybe you meant railties >= 3.0 ?
use "on US-ASCII"
What API is being changed by passing the args to this function call?
Ugh, github was being SLOW. :(
Nope it's not using /news/items instead of /news. I'll see what I can find out today and then we can talk it over later.
Question for the early dynamoDb users out there.... how are you guys handling errors that occur from AWS when you've reached capacity?

Exception thrown: Error: EXECUTE [400]: The level of configured provisioned throughput for the table was exceeded. Consider increasing your provisioning level with the UpdateTable API

I'm assuming you guys have implemented some sort of queue to store items that didn't successfully get added, updateTable to increase throughput, and then start re-inserting them again?
Hmm, I'm writing a test for this and wonder if the output format is really what we want. Right now for each op it creates a new object which is named by the Message's print() output. Should I change that to be something more deterministic?
Nice! I made it gdefault independent: https://github.com/henrik/dotfiles/commit/aaa45c1cc0f9a6195a9155223a7e904aa10b256f
Should this really be an `ArgumentNullException`? The reason I ask is I ran into this and I was confused because the argument I gave was definitely not null. :)

Perhaps this should simply just return immediately since there's no work to be done. Or at worse, throw an `InvalidOperationException` or a more generic `ArgumentException`.
@Ryan-Myers Hey!

As I was examining this code I noticed that the recording thread uses the same stream as the output thread, causing the two steams to collide and space out, making voice incoherent and choppy if both objects are trying to use the steam at the same time.

Additionally, when I plug my headphones into my phone, my voice isn't transmitted to the system. I figured since a headpiece could be plugged into the phone it was trying to capture the microphone from the headpiece (which didn't exist on my pair of headphones) so I wasn't being recorded.

Instead of recording the input perceived on "MediaRecorder.AudioSource.MIC" i'm having the program always choose the phone's built-in microphone that's located on "MediaRecorder.AudioSource.CAMCORDER"

The "You need a camera" bit was, for all purposes, a joke :)
this is not true. can you remove this paragraph?
Very nice to see this "shape up", just to use a groan-inducing phrase :D

At first I thought you had found a way to use a built-in Java resource loading strategy to make the manifesting process work, which I tried at length but just couldn't find the right combination for, then I scrolled down and saw it still there in a utility class format, also reused elsewhere. Yay reusability! And thanks for clearing up the logging, that was on my short list :-)

(Melons are more on the long-list)

Can't wait to see shapes functional in-game!
Thanks, it's crystal clear now. It wasn't obvious at a first glance.
I figured as much ;)
ok awesome, thanks
TODO: add line info to the report. Only be verbose like this when in development mode.
Apparently `push -f` pushes all branches.  Can you fix master and gh-pages?

--Jay

On Sun, Jan 15, 2012 at 11:42 PM, Han <
reply@reply.github.com
> wrote:

> ok, can you reset and `push -f`ed over all my `DON'T PUSH ME BRO`s, can
> you push your `WIP-new-arch` back?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/laughinghan/mathquill/commit/ab16a5b35800211d93032e4157a004a48f8efded#commitcomment-869921
>
i'm in belgium and depiraatbaai.be is not responding.
This isn't a "funky hack", depiraatbaai.be is an official url of thepiratebay in Belgium. Works for the rest of the world too, so no point of failure anywhere.
That file was obviously not meant to be committed
It was a "debug" sleep and I've been going between two different machines so I was moving around using git.  

Here's the deal, I'm losing this battle with distributed notifications because the even though I'm trying to use a singleton that should be on a background thread it never actually receives the response so long as I'm still in limitedOptions.  The moment I leave that method the responses come in.  So in the current state, if you have Tedium running and configured with at least one destination and then you load up the new action screen it will be empty the first time and then contain data if you run it a second time.  As of right now, I'm not sure I can make this work.
check out https://github.com/h5bp/html5-boilerplate/commit/d269a55d4541756c1b339429997e7ca507b5c908?w=1 to see the diff without whitespace changes.

and yeah. the whitespace mod was unintentional. :/
Need documentation on this function.
Does that mean I dont need the bayes_commentlist.js for the dashboard part any more and just use the hook?
Well, I tried but it didn't work. There are more problems inside player.
The one in the environment file, the one you source, there's an echo on the last line that prints the PID of ssh-agent.
You can assume the "type" will be always present.
Just grab the Type and convert right away.
Hi subb
	the uid number what I have sent uuid=999999990019 and
name=Shivshankar Choudhury. which is provided in the aadhar portal as a
Test data and have a look at the response from the server please

Response from the server:
------------------------
<?xml version="1.0" encoding="UTF-8"?><AuthRes
code="1c8d694aaf944511b05d594bc28cc385" err="569"
info="0149951232b1f45f281c7d4f70f3cbbc57c2afd9c0d6bb5f44578bf1304d4868d40000000000000000000000000000000000000000000000000000000000000000100000000000" ret="n" ts="2012-01-09T11:51:31.982+05:30" txn=""><Signature xmlns="http://www.w3.org/2000/09/xmldsig#"><SignedInfo><CanonicalizationMethod Algorithm="http://www.w3.org/TR/2001/REC-xml-c14n-20010315"/><SignatureMethod Algorithm="http://www.w3.org/2000/09/xmldsig#rsa-sha1"/><Reference URI=""><Transforms><Transform Algorithm="http://www.w3.org/2000/09/xmldsig#enveloped-signature"/></Transforms><DigestMethod Algorithm="http://www.w3.org/2001/04/xmlenc#sha256"/><DigestValue>XUd3LPOYdFra0W2jcR+TOzNeDXUjSTJBwWS8CRfEVkg=</DigestValue></Reference></SignedInfo><SignatureValue>DjFrUGfj410No1ymV3PD2lYtMeXyogovtG6HVdSCm/mNosjBG0F5C8rEOVx1wlUP8JvPdvLEmBuX
qheHGKnGa92C5U8e5AV0eGdOuDSOnnIk3/TOYVBlzFPSV0M888BxqXPT
+WuzLwMYIUhiCMOdBSlC
XWXwImFmtCzmAHJJp15vvlAqnmVybekzfFqrYc2YGe0UmS2CSD7IxQEy2Ssi
+RHu/yWOKRwg443x
+ORFe+jCNd4v18ZYMgK8Ltkxf1RhCLWysDcG+2ttlFblZBkJ8Xc6G
+js1w7ZEIOOA6AxQdz63ZYO
CH2ufLgLuqiOrUpByn623uyTh1JTLAeZ3kbgAw==</SignatureValue></Signature></AuthRes>


Thanking you 
Madhu
Apparently the idea came from @dooman, didn't even notice when posting the issue.
#1837
Function name with old nomenclature.
@jpoyau  should we add a test to publish to make sure this runs correctly?
The code style in your class is different then the one used in ME. It's very hard to see what you changed since a space is also a change. Could you reformat the code?
вот эта штука похоже Сиды уронила, надо подправить
$description !== NULL I guess
Hi this is currently the dev branch - which will be 0.6 in the near future.
I don't have something pre-prepared.  But it's pretty straightforward.  For example, here are the lines in my TMV library to create the documentation:

```
texlist = ReadFileList('doc.files')
pdfdoc = env.PDF(target='TMV_Documentation.pdf',source='TMV_Documentation.tex')
Depends(pdfdoc,texlist)
```

Pretty straightforward.
Cool, thanks for this commit Diego.  What's your opinion of this implementation compared to the BDB one?
The way I'm reading this particular line is that this method executes the callback, then passes the result to `repositoriesForUser:includeWatched:page:completion:`. That result is expected to be a block.

(So, as currently written, the callback block is expected to return a callback block, which is passed the retrieved result of the API call)

I think this line *should* be:

    return successBlock_([self repositoriesForUser:aUser includeWatched:watched page:1 completion:successBlock_]);

Thus, the user passes a block in the completion parameter and this callback handles the results of the API call. I think this is what users of the API expect (it's kind of odd to have a callback that simply returns a callback - which is why I think this is a simple mistake).
Co ja mówiłem o static ?
We should set the protection and parent of the alias so that private imports don't break again.
@main-- Thanks. I managed to to guess the same project issue linking, finally. Never would have figured out the commit linking or either of the cross project linking. Now I just need to remember these things.
Removed Bootstrap credit note.
invokes
Hmm, now we just get this:
```bash
<4>[217, WifiService] [  153.699824] dhd_module_init: platform_driver_register timeout
```
I mean stash it somewhere adjacent to the static vxattrs arrays on module load...
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/boss_twin_valkyr.cpp: In member function ‘virtual void boss_fjolaAI::Reset()’:
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/boss_twin_valkyr.cpp:164: error: ‘DATA_CASTING_VALKYRS’ was not declared in this scope
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/boss_twin_valkyr.cpp: In member function ‘virtual void boss_fjolaAI::JustReachedHome()’:
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/boss_twin_valkyr.cpp:174: error: ‘DATA_CASTING_VALKYRS’ was not declared in this scope
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/boss_twin_valkyr.cpp: In member function ‘virtual void boss_eydisAI::Reset()’:
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/boss_twin_valkyr.cpp:443: error: ‘DATA_CASTING_VALKYRS’ was not declared in this scope
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/boss_twin_valkyr.cpp: In member function ‘virtual void boss_eydisAI::JustReachedHome()’:
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/boss_twin_valkyr.cpp:453: error: ‘DATA_CASTING_VALKYRS’ was not declared in this scope
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/trial_of_the_crusader.cpp: In member function ‘virtual void npc_fizzlebang_tocAI::Reset()’:
/home/jordi/server/radical/Radical/src/bindings/ScriptDev2/scripts/northrend/crusaders_coliseum/trial_of_the_crusader/trial_of_the_crusader.cpp:626: error: no match for ‘operator=’ in ‘((npc_fizzlebang_tocAI*)this)->npc_fizzlebang_tocAI::m_uiPortalGuid = 0’
/home/jordi/server/radical/Radical/src/game/ObjectGuid.h:89: note: candidates are: ObjectGuid& ObjectGuid::operator=(const ObjectGuid&)
-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

no mire si algo mas cuando lo arregles lo meto
Re bonjour, Ca fonctionne mais j'ai été obligé d'enlevé les mutlis demandes pour n'en laisse que une, c'est dommage, si tu pouvais m'aider.
Merci
**Part** | **Points**
------------- | -------------
base | 1
advanced | 1
**Total** | 2

```python
del nums[len(nums)-1]
```
You should use [-1]. Python slices are powerful!

```python
if len(even) == len(nums):
	only_even=True
else:
	only_even=False
```
Creative solution!

On section 4 question 4, we wanted just every odd number <100.
Hi Justin, I think that the Rhino based approach for processing CoffeeScript is better comparing to the one using HtmlPage and WebClient, especially after using a GenericObjectPool for reusing rhino objects: https://github.com/alexo/wro4j/blob/v1.4.3/wro4j-extensions/src/main/java/ro/isdc/wro/extensions/processor/js/CoffeeScriptProcessor.java

I'm intending to migrate in the future to newer version of rhino (1.7R3) which should be even more performant.
Да не, не развалится. В худшем случае кракозябры в комментарии будут.
Hey capt did you end up getting time to have a look at this?
Тестирую в Опере 9 - половина функций не вызывается из-за отсутствия вызова каллбэка.
Ah, missed the lack of namespace. I see.

---
Jake Wharton
http://about.me/jakewharton


On Tue, Feb 14, 2012 at 2:59 PM, Johan Nilsson <
reply@reply.github.com
> wrote:

> True, but that's why there's an workaround for it in the styles.xml, it's
> changed to reference it as an internal style instead of using the android
> default one.
>
> Guess the fix is more visible in this [changeset](
> https://github.com/johannilsson/android-actionbar/commit/d1017d79822c3404eb368b1be2f4bd35e00aa332
> ).
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/johannilsson/android-actionbar/commit/7846c4a6fa86f63106037b76a95b0dd4537a33f1#commitcomment-966208
>
On 3/7/2012 5:26 AM, jacob-carlborg wrote:
> Should getsectbynamefromheader be used for the TLS data as well?
>
>

Yes, I just want to make sure the basic idea works first.
same error for me the problem is that the service parameter is not accessible
ok

On 1/3/2012 11:40 PM, Martin Nowak wrote:
> I made a pull for a proper fix to this in
> https://github.com/D-Programming-Language/druntime/pull/112.
>
>
I still don't understand why we have a @minversion and @maxversion.
hmm,you're right, it's currently not used 100%.

Wondering if we should move all profiles to parent/pom.xml, and only redefine the profiles by id where they are used. In that case we only need them defined once.

parent/pom.xml
```xml

<profile>
  <id>X</id>
  <dependencies>
   ....
  </dependencies>
</profile>
```

jms/pom.xml
```xml
<profile>
  <id>X</id>
</profile>
<profile>
  <id>Y</id>
</profile>
```
nitpick three extra spaces
Nice, how is this working ?
No problem, i am glad you solved it!

Have nice day!
this line sends the info off to update the progress dialog
With 796 fixed, this is no longer accurate.
After evaluated to a class or struct reference, an `AssertError` is thrown if the reference is null else the invariant is run.
Ok, I'll adapt my patch tomorrow.  It's worth pointing out that the new getVIrtualFunctions will do exactly the same thing as the new getVirtualMethods once this change is made.
Cara, não podemos colocar lixo no código. Por favor, faça apenas o PR do primeiro commit.
This could just reuse the `index` variable available in the method.
Removed the static property.  If desired, mousewheel can be disabled via scrollview._bindMousewheel(false).
Sure. I'll account for this in the next broken pull.
ok, here it was
yes, the original case was only `case Terminated(_)` which as I understand it is received when it is time for the system guardian to stop.
Now I have also added watch of the hooks, which is the `Terminated(a) if terminationHooks.contains(a)`, i.e. the hook is terminated before it's time to start the stop procedure of the guardian.
I can add a comment about that.
I'd remove this method and use remoteSettings.LogRemoveLifeCycleEvents in the notifyListeners-method directly.
you are of course right
Same comment here as above, any solution needs to be based on non-mutable state.
My intention was not to suggest that you should add toString, it was an example scenario where we would add a method, and break serialVersionUID. We concluded at the dinner the other day that there is no such thing as a perfect crystal ball :-)
Yes, that was what I said.

/Patrik

8 sep 2012 kl. 15:26 skrev Helena Edelson <notifications@github.com>:

> In akka-cluster/src/main/scala/akka/cluster/ClusterMetricsCollector.scala:
> 
> > +
> > +  /**
> > +   * Samples the metrics for the node and updates metrics statistics.
> > +   */
> > +  def collect(): Unit
> > +
> > +}
> > +
> > +/**
> > + * Metric monitoring of the self-node intended for load-balancing / workload distribution.
> > + *
> > + * INTERNAL API.
> > + *
> > + * @author Helena Edelson
> > + */
> > +private[cluster] class ClusterNodeMetricsCollector(val environment: ClusterEnvironment) extends Actor with ClusterMetricsCollector with ActorLogging {
> Actually I can not change to this strategy yet, cluster actor name is not unique ;-)
> 
> private[cluster] class ClusterNodeMetricsCollector extends Actor with ClusterMetricsCollector with ActorLogging {
> 
> val cluster = Cluster(context.system)
> 
> So we can refactor it when Patrik's PR is in master.
> 
> —
> Reply to this email directly or view it on GitHub.
>
Great. Probably much more probable to be right. No weird rounding errors.
same here, and possibly on other places
Creating the set is probably cheaper compared to the getNodes + await though
please make it into one sentence: To use ``DurableMailboxDocSpec`` add this dependency::
`val` generates extra bytecode (not as efficient) as without the `val`
`private[this] val` generates same thing as without `val``

I didn't hear any more comments regarding my objections to `private[this] val vs. leading underscore`.
This still is wrong...
yes, if there are no metrics for a node
Turns out signup emails were not tested, and now they are.
Note: `ViewModel::$glob....` instead of `static::$glob...`
remove the `/` as `pygwt.getImageBaseURL()` will always have a trailing slash (`/` also breaks relative urls)
A further step is to use read_attribute here and deprecate accessing trunk_id or tag_id

Better is to do the migration and add both semantics to wql.

Want me to add that here or wait?
Where is the progressive frame? fields[0] should be the progressive scaled image.
Sure, but it assumes consistency in the way codec handling in Python is done.But I agree with you we don't want to test every other codec out there. Let's just trut Python's test base.
Там в требованиях хотят чтобы выводился именно оператор заказавший услугу.
Для старых кейсов этот код работать не будет, т.к. из case.actions закрытые действия удаляются.
Я вижу два способа:
 - `select * from actiontbl join servicetbl whre actiontbl.parentId =  servicetbl.id and action.type = "xxx"`
 - при закрытии действия "заказать услугу" вписывать оператора прямо в соответствующую услугу.

Второй мне нравится больше (меньше запросов к БД), зато там лишние функциональные зависимости появляются.

Btw, мне кажется, что это немного связано с #272, с поправкой на то, что история не только у кейса может быть, но и услуг (и т.д.).
Is the sleep needed?  It just slows down the testsuite.  If it is, can we think of another way to do this via polling?
_globals?
`packagedJS` is now an unused argument to this function now. Might be able to drop it from the `WorkerClient` initializer.
The most elegant solution would be for all platforms to implement the standard setImmediate so none of this song and dance would be necessary. Related issue here. https://github.com/vert-x/vert.x/issues/448

Diving into the Java packages namespace is not acceptable because touching those names would initialize the JVM in some browsers. That is, unless we find a solution for that problem as well.
only Nette.
maybe `!$def->class` missing in expression
Je.
Can then do

    fp.write(lines)
    fp.write(polytemplate.substitute(...)
FeatureEditorGrid is a special plugin, because it inherits from an Ext.Component subclass. Having said that, you can give it an xtype (and register it as component) in addition to the ptype. We haven't done this before, but I think you can use the same string as xtype and ptype.
if (!propertyString.isEmpty())
все это будет работать в английской локали?
yeah, 404 on recordnotfound is standard and without argument against can't imagine having an issue with it :) :)
`response.url.should include(forum_topic_path(topic.forum, topic)`, I think.
In this commit - https://github.com/rails/rails/commit/336ff8a97e9391d4111e923a9b841376110d04c9

the code is splitted in two files has_one and has_one with that condition so IMHO is the same, in some later commits we merge that code in an unique place
Another thing. I didn't remember now why we need to delete. Do you know?
I don't know, seems weird to me.

```ruby
case value
when Array
  convert_array(value)
when Hash
  convert_hash(value)
else
  raise
end
```

This reads better to me. It is cleaner. It make clear what are the supported types of value without need to read the whole class.
:scissors: extra spaces before `do`
that's... that's it? nice
Ah, you're right. So `fjbg.jar` is not retrieved by `pull-binary-libs` anymore?
`receiver.toString + args(0)` ?
import numpy (as np) ?
`callback` could not exist, and this would crash the program. Needs to be checked.
I am basing each next branch on the previous one. So at the moment I
have unpublished branch 3 that is based on this one.
I did not find other solution to change the system for all units and operations without specifying it at each time. But it was not intended to be use only in interactive mode.
while to for loop: `for i in range(len(terms) - 1):` and then the `i += 1` can be removed below.
Auch wenn es korrekt ist, "als Quellsprache zu belassen" höhrt sich ,meiner Meinung nach, etwas gekünstelt an.
"Es ist sinnvoll Englisch als Quellsprache zu verwenden"
Yep. event Namepacing.
This replaces the existing part of the topic? is this desirable or should it insert, pushing other topic stuff on. We'd need to provide a delete command but currently we'd have to do lots of setting position to '' to simulate a delete and that might end up with a topic like:
Very good!

Beside few syntax fixed the code styling is good.
0.5 could be anything fwithin the correct answer choice from 0 to 1 not 0, since we are essentially calling the validation with these temporary values to get the validation to not hang up on the 0 for wrong.
I think that we get around to implementing tool tips, we should follow a unified approach, so whilst this tip is good, I think I would leave it out for now.
I think this behavior is slightly confusing - it would probably better if it were more explicit about the fact that content retrieval is not supported, since returning "Expanded detail not available" leads you to believe that content retrieval is actually supported, but there is currently no detail available for the chosen id.
Do we still need this level of logging for spring?
"pre-indexed" is not a common term that we use in relation to MongoDB. what is the concept you're trying to convey here?
not :option:
just wondering out loud about putting the common copy in individual text block but it's more work than it's worth.
Space between `#` and `T`
you could use : np.array(map(float, iop))
This a leftover?
Not too sure about this. In the past, it has either been one long if statement or it has been set in a variable beforehand.
@naderman It's optional and the default is false?
As before, this should default to false, I think.
Should be `3.0.11-RC1`
I don't have a strong feeling either way

p/phpbb3#28
- is a typo
There is an added check that needs to be done here, however. Specifically, we must check that all the PolicyLabeled values are the same for both documents. (Old, but maybe useful: https://github.com/scslab/hails/blob/7d0ddda4723ca4b79e335a9c812292c6e1ce37e9/Hails/Database/MongoDB/TCB/Query.hs)
This is a list of the ENV vars, not the class attributes.  So it should be capitalized, right?
учудващо обаче:

```  ruby
  render 'activity_content.html', :activity => activity
```

сработи.
oh god how did that get there I am not good with computer
:) will fix
Engine при создании передаются настройки. Передавать ему imageServer неправильно.
I don't think we do at this point, so this can be front()
@iavael , how would you modify the patch to improve the situation?
it is already somewhat confusing with the set of Confusion* things we have... may be it could/should be more descriptive and stay in-line with the others, e.g. ConfusionMatrixDataset or ConfusionMatrixNode?
namespace should be 'RaxaEmr.Registration'
again, a shared constant would be helpful, so you just have to update it in one place
will this ever fail to find the drug? 

If so, any bad outcome if  ```Ext.getStore('...').getAt(null)``` or ```Ext.getStore('...').getAt(-1)``` (and likewise setting the component's values to null below)?
这个文件已经被我删掉了。不需要提交进来了
I just made a new commit that fix this.
Worth asserting stmt.statementParamJavaTypes.length == params.size()? Seems a little odd to get the size in 2 different ways, or am I misunderstanding something?
What does filter = filter mean here?
Does this really want to read an entire file into memory just to get its first line?
As implemented, this function could be expressed more concisely as "readFileIntoArray(srce)[0].rstrip()".
Use |safe instead.
Likewise here as above with platform comment.
Are you aware of the result of this expression, depending on the value of type ?
It's on my TODO, I have to write it
This whole function should be enclosed in #ifdef HAVE_LIBXML2, so it would disappear completely in libxml2-less builds.
I might be wrong -- shouldn't it be an error instead?
Because this causes your non-existent-table errors below... You delete the non-auto M2M tables once as M2M and once as models.
Removed in 7e8c64d8e68398937028400f801dbe02a830f9d4.
I'm want to focus on the Django. I think this should be something raised in an article outside the tutorial.
Just to be clear, you were thinking along the lines of the following, correct?

```objc
return boolean.boolValue ? kCFBooleanTrue : kCFBooleanFalse;
```
The alternative would be such:

``` JavaScript
plugins: [
  {plugin: "./path/to/plugin', overrides: "hosts", opts: {}},
...
]
```

Personally, I much prefer what is there now because it is clear what the parameters and options pertain too.
* line:28でwarningが出ています。対応してください。
  * 'presentModalViewController:animated:' is deprecated: first deprecated in iOS 6.0
What's the performance effect of this? My gut feeling is that a virtual function call is more expensive than an inc instruction.
@xiaomonp @rcrichton No, I believe it's fine as is. We're logging the fact that we ran a XDS Stored Query request.

However, we have to indicate what the outcome was of this request, successful or not: so @xiaomonp the outcome variable needs to get set accordingly. I believe there should be a status field in the response object somewhere.
why not just `ret[i] = posX[ i ] + " " + posY[ i ];
OK, I just realized I'm thinking internally. You're saying that users could pass a nodelist to jQuery.unique. That is possible. But if we leave this, I don't think that argument goes with the setFilters loops.
Looking at the result of install_test(), the result table, extraction has duplicated rows, which comes from duplicated rows in viterbi_mtbl. I tried truncate for viterbi_mtbl and viterbi_rtbl at the beginning of this function, and it seems kind of fine, but the process for document 3 results in only two terms, "his" and "actions" and rest of words disappeared.
you mean wrap them in a single pg_try?
maybe just make it:

```js
if (ModalToggle && modalToggle.hash) return document.querySelector(modalToggle.hash);
```

and remove line above
Agreed. What about adding a new [section](http://meteor.com/examples/leaderboard):

## Things to try out

1\. Inspect *Todos* and *Lists* on the browser console:

```js
Todos.findOne()
Lists.findOne()
```
2\. Inspect the saved data on the meteor mongo console:

```js
$ meteor mongo
db.todos.find()
db.lists.find()
```
It seems that `encodeURIComponent(null) === 'null'`, which makes me even more curious why we need this check. Was something not working with `null` the way it was before?
Fixed it.  Repull.
Needs to be a list rather than a string to avoid quoting issues; also can use `#{etc}` and `#{var}` which are rooted in HOMEBREW_PREFIX.
Maybe the option should be named "--with-html-docs" or something?
`if build.include? 'with-metis'`
Oh, you're right. Misread.
domains = rest_client.domains
this should be OK. getSourcePathofSharedFile() uses the path which is added to the Shared folder. Therefore the path needs to start after '/Shared/'.
I think it's more clear this way.
Hier würde ich nur "dir3" erwarten, hätte der Ordner noch Dateien, die auch. "dir2/.." sollte rausfallen, auch wenn es einen Unterordner "dir3" gibt.
Indeed it does, my mistake, forgot a `?` in there.
Pretty sure I was thinking that was just the thing to do so I stuck with it. Is there a situation where that's suitable (assuming this isn't it)?
Capitalize first word. Also, does this need to wrap? We either impose 100 or 120 char columns (I can't remember).
Include an exception message on this and the line above?
My usage preference is assoc args, so it makes sense to me to include it. Updated in 7865cc0
这些位置都需要root权限，而且属于不可持续的方式，即，你这次即使暴力改了，下次更新R之后可能就又忘了改。之所以在用户目录下改，就是为了避免这两个问题。我的原则是，一切能在用户目录下解决的配置问题，绝对不动用root权限。
What would be the use case for that?

You could write `withStacktrace(nullValue(Throwable.class))` for that purpose.
Could be shorter and cleaner:

    background-position-x: if((type == 'home'), 5px, left));

Also, a code style nitpick: we favor single quotes over double quotes whenever possible.
Added some comments around these related changes.
Typo: please replace Datein by "Dateien"
Funktionen und funktionieren in one sentence is too much, please replace funktionieren by "ausgeführt".
If this doesn't return, then the error message needs to change. Probably easier to return :)
I don't understand how this runs for each test. Is this comment incorrect?
I still think it's weird to use `isDirty` for both these cases, because it makes the code here hard to understand--we're using `isDirty` to mean two different things in two different if statements, and it's not obvious just by inspecting this code what the distinction is.

If I'm understanding this correctly, the `isDirty` we're declaring in this function (and that the DirtyDocumentTracker is tracking) really means: the user might have changed this document since the last time we checked, either by editing it in Brackets or by editing it in an external editor, but we don't actually know whether either of those things are true. So it's really something like the "DocumentMightHaveChangedTracker" :) (And, in fact, we do need to track `change` events instead of the `dirtyFlagChange` event, because we want to find out whether the user has made changes since the last time we checked. So `dirty` is a misnomer anyway, since we really don't care about tracking the value of the dirty flag per se.)
I don't know how to figure out what git commit hash svn 3229 references, but it would be nice to update that information here.
I swear I already made this comment but can't find it. What's this change for though? If this function fails it should do so with an exception not a boolean return. Plus the default case is to return None which seems like it's not what's intended.
It'd be nice to add the -moz versions where possible. How does it look in Firefox?
ah, did you modify a bootstrap plugin directly at some point? The best practice is for any new functionality to create a custom bootstrap plugin that builds on an existing one... anyway, no biggie
this looked like it was fun to generate :)
This method, and the two constructors, must be documented explicitly.
Thats true.  Guess the only arguable benefit would be a likely negligible performance gain.  Doesn't look this fix is going to be merged anyway ;)
The it still is?

I am confused do you want it in this order:

    import mongoengine
    mongoengine.connect('settings')
    
    import os.path
Don't hardcode next, but get it from the code. (We have a global context processor to pass such values to templates.)
Might it be easier to just listen to `window.onmouseup` here instead of using `trackevent.listen('trackeventmouseup')`? If you mouse down on a trackevent, assigning the listener here would just let you capture the trackevent in a closure instead of using any member variables, right?
Do we need the padding reset here? Seems to me like this probably could be omitted.
Same deal here, we want an async event instead.
this may result in a double increment of the iterator when an entry was deleted
My pep8 checker complained: 
"Inline comments should be separated by at least two spaces from the statement" 
(http://www.python.org/dev/peps/pep-0008/#inline-comments)
add a copyright statement?
Tämä pois
This is quite ancient definition:

    ...WriteConfig () ``{

you can just use

    ...WriteConfig () {
change because of formatting change due to removal of an if-else loop
We can, fixed :-)
is http only one option? can someone use ftp?
I would rather check if it does (not) match regexp 
^[a-zA-Z]+://
If you insist ;)
I think we use "katello" as a oauth key, could be a sane default.
Same for this class name.
This is as well.
It was a bad merge somewhere in the past.  I was probably the one responsible but basically the code was duplicated.
Try running "find . -name '*~' -print0 | xargs -0 /bin/rm -f" in your gaia directory before generating appcache manifests.
Does this logic apply to Pinyin also?
Not needed - see e.g. gmakeng
This should be a relative import - also, I think for consistency, this should be imported in the methods/functions that need it (and not in a try/except). At the moment, if the compilation of sofa_time fails for some reason, the user would be left with a confusing message when trying to call methods/functions.
I think it is prudent, we really want to avoid death spiraling, and printing messages about yourself only seems like it'd add to the problem.
Is this loop really necessary? Can't you use the recursivity of parseChildren() to look implicitly at the theme children?
the curly brace should be on its own line (PHP-CS-Fixer FTW :smile:)
@stof some API generators require this. It doesn't hurt.
Add a line break between attribution and assertion
Removing the ``getEntity`` method is a BC break
Yeah - would make more sense. The has_closing_date method is true if close_date == nil though. Can change that so it returns as expected.
This indentation needed? :)
Alternatively we could treat this as:
```
    def __next(self)__:
        # ...

    if not py3compat.PY3:
        next = __next__
```
I'll make a new one, as it seems best to minimize the number of PRs around?

Tim

On 27 March 2012 18:11, Thomas Kluyver <
reply@reply.github.com
> wrote:

> > @@ -403,6 +416,16 @@ def _greedy_changed(self, name, old, new):
> >          When 0: nothing will be excluded.
> >          """
> >      )
> > +    limit_to__all__ = Enum((0,1), default_value=0, config=True,
>
> If you've re-forked the IPython repository, you can do
>
> ```
> git remote add takluyver
> https://takluyver@github.com/takluyver/ipython.git
> git fetch takluyver
> git checkout drtimcouper_all_
> ```
>
> Then either make a PR against my copy, or make a new PR against master (in
> which case I'll close this one).
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/1528/files#r610236
>
Hm, I possibly shouldn't have committed that line - I forgot this test was a generic EC2 one. But, the test does depend on a specific AMI naming pattern that AWS EC2 uses... I'll move the class shortly and update the pull req, unless there are any more comments.
It's still necessary. I'll give it a better name, though.
Frankly, instead of adding this function, I think we might as well just get rid of the `g_opts` static `git_checkout_opts` and declare the opts on the stack inside each checkout test function.
match an open curly
I believe that there is a similar error (my fault at https://github.com/urish/jvsthost/blob/5679b6840930b7a7921bd11139725a23ba401060/src/com/synthbot/audioplugin/vst/vst2/JVstPersistence.java#L102). Would you please also correct this?
Linebreak long lines on occasion.
I don't know.

@jlongster : Do you have any idea?
the `<acronym>` element has been removed in the HTML5 spec, use `<abbr>` instead http://www.w3.org/html/wg/drafts/html/master/text-level-semantics.html#the-abbr-element
I think this reads better as:

    locked = thread(is_locked=True, save=True)
    u = locked.creator
    p = forum_post(thread=locked, author=u, save=True)

However, that doesn't affect the actual test at all and could just be me.
looks like this case never gets tested, need a test for it to get back to 100% coverage
this is a really interesting thing.  The default behavior of a python program that receives a SIGTERM signal is to raise the KeyboardInterrupt exception.  So rather than invent something new, and because I want essentially the same behavior for SIGKILL, I just unify the signal handlers.  

An interesting aside, if you do not register a signal handler, you cannot predict which thread will get the KeyboardInterrupt exception.  All threads must be prepared to handle the signal to kill an application.  I got in the habit of coding for that possibility.  Once you register a signal handler, however, then it will always be the main thread that will run the handler interrupt.   Since the other threads are already coded to handle KeyboardInterrupt, I just exploit that code to shut down all threads the same way.
This was my code and I apologize for this. Thanks for making it better.
Would this not fail due to previous calls to `Sidekiq.hook_rails!` ?
If I get it you don't need to load repos here, because you need only locally installed locales.
I think it would be nice if you can read it from method, so in this case there could be method like
load_system_metadata
and in previous case
load_all_metadata
so difference is quite obvious.
I wasn't aware of entry_points,  the method I used was based on docs from disutils (http://docs.python.org/distutils/setupscript.html), and have used in a project that runs cross platform.  I will dig more into entry_points though, thanks for the heads up.
no, I mean such as using a method of an object as callable (which would fail on PHP 5.3 when calling it this way but is valid when using ``call_user_func``)
It does. This method can't be called before a resource was loaded, and resource loading cannot terminate unless `loadResourcesFromTheme()` was called at least once, where the template is set. I'll add a comment though.
@asm89 you cannot really have required parameters after optional ones. It will simply meant that the previous ones are not optional anyùore
it's Boolean
HttpFoundation is only an optional dependency (just like now for the ``RequestContext#fromRequest`` method). typehints don't trigger the autoloading, so HttpFoundation will never be required if you don't call the method
this behavior was wrong imo, foo1 should return the command named foo1:bar as it's the only one starting by foo1.
Yes, regards this http://developer.github.com/v3/#authentication
GitHub supports only plaintext login\password and OAuth2 for auth
I am confused. What's the difference between using a callback and using the Constant.JAVA_LIB_PATH directly? The comment above seems to indicates that the correct value relies on some objects to be created later so you need a callback. But then the implementation of the callback simply returns a constant instead of getting some fields of an object that the target depends on. So I did not get the point of using a callback here.
networking errors.. always useful to at least NSLOG imo
Should it be a warning or info?
Nice, I didn't know we were starting to move to this pattern. That means we're in a position to implement conf.d-style custom checks.
You MUST irc_lower this, as irc_lower does more than just make the string lower case.
The ```defined(_MSC_VER)``` sound good to me. The ```YieldProcessor()``` should be inside the loop to have the same functionality:

```c++
#elif defined(_MSC_VER)
        for (int i = 0; i < delay; ++i) {
# if defined (_WIN64)
            YieldProcessor();
# else
            _asm pause
# endif
        }
#else
        // No pause on this platform, just punt and do nothing.
#endif
```
On second thoughts, I'll take out the non epsg string.
This could be sped up. @rhattersley do we not cache the boundary polygon somewhere in the CRS?
Think of player as an added level of abstraction basically. SpoutPlayer implements Player -> Player.getEntity() -> Entity.getController()

An entity can have a PlayerController as their controller (ie. VanillaPlayer) but Player is not actually a controller itself.
Could this be changed to something like ``target.respond_to?(param) and not target.send(param).blank?``
make this readonly
If ```buf``` continues to exist (see above comment), would it make more sense to do:
```python
    self.buf.write(line[idx+1:])
    self.buf.write(buf)
```
instead of doing the string concatenation?
OK. I assume that's a known thing in Quantum, and doesn't need documentation here?
before we start importing we put a message in the activity feed saying we will attempt it. this is helpful to track down unsuccessful imports (ie where we still ahve bugs) if we have some remaining import bugs in production.
@bstansberry  yes it could be differed but implementation would be much uglier :(
my first impl didn't use it...
В PostgreSQL 9.0 это не работает, там как-то более извратно тоже самое делается
small thing, but how about:

       for key in conf.keys: # key is something like 'large', 'medium', 'small'
            _save_thumbnails(image, path, conf[key]['size'], conf[key]['suffix'])

this will mean that if a person adds another thing to conf (lets say 'supersmall'), that thumbnailing will automatically run in the future. its also a line shorter and has three fewer hard-coded constants :)
There are some indentation issues in the snippet above this comment
I don't understand this name.
sorry, should be `textContent`.
I think I already made a comment about this last time but maybe not. Don't forget to use the l10n api.
FTR, you don’t have to add these strings to other locales. The en-US locale is always loaded first, and now I have a lil’ script that inserts those commented line with a nice `TODO` mark.
In `ThreadUI` please!
heroku is the site where the page that extracts the access_token and pass it to the parent app (from the window) is hosted. This place will have to be replaced by a host in the production environment. 

To avoid problems I have created settings to hold all the values related to OAuth-Facebook configuration, so heroku will no longer appear on the code. 

Thanks for spotting this problem.
This will be very complicated to localize and this should be probably avoided.
DataView is available now, so we can probably get a performance increase by modifying this code to use it.  It might be nice to make that change and contribute the patch back upstream.  Not really a high-priority, though, especially if we end up getting metadata from gecko through the audio element.
again, if the value is changing we shouldn't keep the `kMaxNumberOfDigits` name, but change it to something without the `k`
classList already have a toggle method :)
The `<button>` element actually has a `disabled` property, we should use it.
Rename this object to `FakeAlarmManager` so people would know
and filters needs to be quoted for legal JSON.
I haven't tried it out, but from my own fat finger experience typing on mobile keyboards, I would think that this kind of error would be more common (and thus higher ranked) than transpositions
This line should be `self.turnScreenOn(true);`
Actually, no: it's slower. Or at the very least won't be faster. Most optimization articles will recommend removing tag bits on id/class selectors unless the class applies to move than one tag in a separate way (which is confusing and not recommended): https://developers.google.com/speed/docs/best-practices/rendering#UseEfficientCSSSelectors

The CSS doesn't need to know which tag it applies to and although I appreciate explicitness, I don't really think it is here. Just extra verbosity here :-)
原文だと
    
    The output above shows the "strongbad" node currently has an empty tags property: `tags: ''`.

訳は、

    上記の出力では "strongbad" というノードに現在は空の tags プロパティ `tags: ''` が付いていることを示しています。

と最後が ": " で終わってないけどここはいいかな？　
    
    上記の出力では "strongbad" というノードに現在は空の tags プロパティが付いていることを示しています:  `tags: ''`

とした方がいいだろうか。個人的には前者でいい気がしていて、前者の方で直してる :wink2:
can be changed to a one-liner:

```getTimeForUserId: (userId) -> run.time for run in @runs when run.user_id is userId```
I think you forgot to change the name of this
This should be a val, right?  Isn't it always 0?
Make an issue for this on GitHub.
The error message should be much more detailed than this. Include the reason why the app couldn't be stopped.
Agreed, this part needs to be fixed.
```coffeescript
o = {adjusted data...}
switch board
  when someboard
    url = Redirect.path '//ar.chi.ve/', 'fuuka', o
```
Yeah exactly, you should never be allowed to chat in more then one channel at a time.
Would you mind converting this to a normal if/else block so we can preserve our backward-compatibility with old Python versions?
Should virsh.freecell_options be virsh_freecell_options?
Because...I forgot. :)
any reason not to just transmit the patchlevel as an int directly?
I don't think I had added this, but moved it here from another file. This must have been removed from the cocos2d-x in the meantime?
incorporate dependencies from a VCS? This does not make sense to me. Can you clarify what you mean by this?
Yes. The try ... catch is not necessary. The best documentation for the test is obtained if there is an explicit throws declaration for both the IO and the SAX exception (rather than the general Exception).
Honor qutIM conding style conventions, fix brackets
Hmm.. that method was almost fully rewritten a few times (I did not do the coding), but in the end the bbox part remained unaltered, so I guess we need to restore it
If any kind of serialization happens, it should definitely be up to the driver to handle that. What is a different driver going to do with a YAML string?
Why not call this `onClick`? As it is, I have to remember what a "trigger" is defined as.
No, I imagine you are correct.
If you think is confusing, I can just remove these comments. I wanted just highlight that an empty mime type / parameters is allowed, but has a specific meaning for the browser.
Do not overload an argument variable, especially when the type change.
Looks like `Tab` constructor in FF module takes options as argument, I think it's a good idea to be consistent here too.