labofsahil/github-event-dataset-2012 huggingface dataset:
The first iterations of h5ai had this line, and it brought up some problems, since this module is named differently in some environments.

I finally completly removed this check so the user gets error messages if something is wrong.. (no autoindex enabled)
I see, more to learn. Need to edit file from shell, then use Git command line.
No probs! :)
You should probably add `$conn = null;` in your `catch()`-block so the variable is set. Can you update your pull request?
Hi Dan, 

Ive merge the changes into my fork and the tests are now passing. I wonder if there is a policy that is machine specific preventing the loading of those files without a password?

Anyhow, all green for me now.

Cheers,
Byron
Oh, and this is a tablet I'm using (in case your device is different)
Ok,
Removed the double slashes, because I have no clue what it is doing. Here's branch with the new commit and this commit:
http://github.com/bradymiller/openemr/tree/security-exploit-fix_2

it is testing well
Similar to how Calendar and Story Budget have help both on the module settings page as well as the actual module view, should we add some help text to the Post edit page for editorial comments (and perhaps also Editorial Metadata)?
Thanks! I think 99% of the changes in this commit are tabs/spaces and carriage returns.
Hi.
CommandBin.java is Do not use [org.cain.cmdbin.commands.*;] ?
I did do this before, but somehow my initial patch here didn't carry those changes over.
Значение сдева должно быть равно содержимому array. Подставь его вместо __ и убери size
GUI和sqlite有关的样例代码
sad to see go private but its a stupid mode to say good bye....
Carsten-

I just tried TortoiseGit - I couldn't find any way to have it do the equivalent of git diff origin after doing a git fetch.

Also,  it made my system churn - perhaps a conflict with my TortoiseCVS and TortoiseSVN installs?

-Tom
Oops, just occurred to me that this line can't be here.
These are the one I did in PSDB. Feel free to use them for UDB. You'll have to change the guids. I'm sure you knew that. Project-Silvermoon was shutdown by admin, But, I'm still keeping the DB alive... :)

DELETE FROM `gameobject` WHERE `id`=176360;
INSERT INTO `gameobject` (`guid`,`id`,`map`,`spawnMask`,`phaseMask`,`position_x`,`position_y`,`position_z`,`orientation`,`rotation0`,`rotation1`,`rotation2`,`rotation3`,`spawntimesecs`,`animprogress`,`state`) VALUES
(23620, 176360, 329, 1, 1, 3500.58, -3294.82, 132.417, 4.55695, 0, 0, 0.759872, -0.650072, -180, 255, 1),
(23619, 176360, 329, 1, 1, 3567.9, -3352.45, 132.02, 2.31857, 0, 0, 0.916517, 0.399995, -180, 255, 1),
(23618, 176360, 329, 1, 1, 3663.38, -3630.41, 139.845, 2.20469, 0, 0, 0.89227, 0.451503, -180, 255, 1),
(23617, 176360, 329, 1, 1, 3660.19, -3475.45, 139.689, 4.82007, 0, 0, 0.66803, -0.744135, -180, 255, 1),
(23616, 176360, 329, 1, 1, 3651.15, -3164.58, 129.454, 5.34443, 0, 0, 0.452331, -0.89185, -180, 255, 1),
(23615, 176360, 329, 1, 1, 3681.64, -3402.14, 134.207, 0.670026, 0, 0, 0.328781, 0.944406, -180, 255, 1);
Yeah configuration over convention, I agree. An option to set a _global_ route for an application is a must have because for now all the routes have to be defined three times.
Can we include this conditionally so that other browsers will still benefit (including IE9)?
Thank you! One small issue, probably more related to vim than to this commit. I have `set pastetoggle=<F11>`. Now whenever I press F11 in normal mode, the statusline is not updated instantly (only after cursor movement). In insert mode it works as expected :)
Is there a documentation somewhere about the actual allowed syntax in shebang lines? Until now I've just been perusing the wikipedia page and some sample scripts I've found here and there.

Do you mean something like this?

    #!:

instead of

    #!/bin/sh
please revert these commits.

I won't accept patches that remove examples.
Yeah thanks but now alot startup errors, i hope is the right rev xD

http://paste2.org/p/1912443
And here I was thinking typing out 'Catastrophe' all the time was annoying...
We were using the proxy objects because of compatibility with key-value observing. Mutation of the entire collection would not fire a change on the observed collection keyPath. This may well have changed with iOS 5 -- I haven't looked into it.
Let's leave it as is. Eventually we will merge the Svg-based GraphDisplay into the Graph, and the distinction won't matter any more.
but that means strings will always result in 3306, which is wrong for pipe names that are strings
Opened https://hibernate.onjira.com/browse/HHH-7151
Ah sorry, I haven't seen that.
I'll remove this.
Should this be changed to elseif?
This change pretty much kills the blog engine administration.  I'm happy to try and help fix this if you can give me some guidance as to what the overall plan is.  I get a nil object for @posts when the template is rendered so I'm guessing this is pushing something in crudify out of scope.  Do you have a timetable for finishing these changes and are there tasks I can help with?
I couldn't understand this code until I actually run it

I would rather have something like:

```js
var stream = require('/.');
stream.on('*', function(package) {console.log(package.name+': '+package.description));
```
Una pregunta mobiledev este emulador es para la version  4.3 
¡¿??
Good catch -- chunksToPortals is actually a local variable, but worldChunkPortals isn't. Changing to "this.worldChunkPortals"...
Oops is right :)
That's a touching attention to details :). It's great.
After some discussion we decided to remove any automatic error suppression that hides errors with es, see GH-45 and GH-60. All es requests pass through this bundle's Client that extends Elastica_Client so that can be overriden with a version that catches exceptions as required. Some info on this has been added to the readme file on this. At the time the try/catch blocks were added to these the Client class had not been added so there was not the single place in which es error could be caught.
There are some issues with the port.

 * X11.Xlib should do a _public_ import X11.X
 * ```_Xconst char*``` in X11.Xlib should become ```const char*```
 * enums in X11.X should not have a scoped name (see KeyMask)
就是这个commit弄得灯不亮，不过这个不是要紧的事情，先别理
Did you test this? Seems like this conditional isn't closed.
Is the purpose of this commit to make the kernel compatible with newer and more ICS-friendly EGL libs? If so, where can those libs be found?
U tuned interactive governor or u did it coz it wasnt compiling?
i can't seem to add any CPU governors *errors*
what about the replaced asset_path? this should lead to a broken image?
I'm probably wrong here but shouldn't it be ```self.treedepth``` instead of ```p```?
Since ```p``` was defined in the removed row: ``` for p in xrange(1,33): # max 32```?
What would you suggest? I wanted to keep things aligned with what they are
called in the menus but if it will break things I can revert.
On Jan 18, 2012 3:20 PM, "nebkat" <
reply@reply.github.com>
wrote:

> I think they most use Settings.LOCK_HOME_IN_MEMORY rather than the string
> directly...
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/DvTonder/android_frameworks_base/commit/64885cd8eaed5575c96adf13bddcd96056afd1e6#commitcomment-879451
>
what results have this patch?
Many users of my ROM still report side swiping notifications can hanging the phone sometimes & needs a reboot. Our phone is the Motorola Triumph. I use "clear" and never have freezing problems myself, so something's still causing problems with side swipes.
trailing comma here?
do we need it here? can you avoid unnecessary code in html?
redirect 2?
@norihiro-w The only reason I can imagine to use is for debugging. In my opinion we can remove it.
I'd like to see one attribute change event handler used for all of the temperature attributes. The event handler can inspect the attribute which has changed, and apply the conversions appropriately. Then the event subscription can look like this:

```javascript
this.after([
    'fahrenheitChange',
    'celsiusChange',
    'kelvinChange'
], this._afterUnitChange);
```
I would like to ask that you enhance the validIn and invalidIn to be not just a single string, but an array of strings, similar to the allowIn, allow and deny properties.

That would allow for handling conditions when a property is valid/invalid in more than just one possible parent.

For than matter, maybe we should just continue to use the "allow" and "deny" property names, applied now as well to htmlAttribute properties.  Helps to keep the understanding and expectations consistent across all usages.
`this.configuration`
shouldn't currentTransport be used?
Wow, thanks, I was not aware of that. I have to look at CARP.
What's the rationale for having two OrderedMemoryAwareThreadPoolExecutor ? Do the settings make sense?
Looks like a no-op to me
currentNodes is not used
I think that is only needed for 0.12.0-RC1. Just like the doc says.
Variant view for Json renderer.  I think I need more of these.  For pointers too, probably.
this is exciting!!
Why not `type = need_type ? ...` ?
One sentence summary missing.
```
grid will be ``(16, 16, 16)``.
```
Maybe needs a little example to explain what an item of a subscription is
For PEP-8, 2 blank lines here
> In src/com/sun/jna/NativeLibrary.java:
> 
> > @@ -124,6 +124,12 @@ private static NativeLibrary loadLibrary(String libraryName, Map options) {
> >          }
> >          catch(UnsatisfiedLinkError e) {
> >              // Add the system paths back for all fallback searching
> > +            if (Platform.isMac() && libraryName.contains(".framework/")) {
> why the trailing slash?
> 

It seemed more consistent with other tests in the file, since they could all be tested with `beginsWith()` and `endsWith()`.  Using a leading dot and trailing slash bounded the test on both ends. And it fits the definition of a Mac Framework: a bundle directory structure with a `.framework` for the directory name. Thus the test includes that the libraryName acknowledges it as a directory.
We should never get a trashed card by id, so you can test and throw an error, but this makes it a faster search.
This can easily be tail-recursive... if sequence was a List[Long]
Learn https://help.github.com/articles/ignoring-files (global gitignore).
GLOBAL STATEMENT? No :)
Doh. Yeah, you're right. Um, this has been merged so I guess I'll open a new one tomorrow.
`method=self._update_method`?
Was not rely sure what to do with the error so therefor i skipped it.
Thought of it some more and a suggestion how to handle this is included in later commit.
If it isn't, make it so and remove the line above
лишні пробіли не став `alert(     "error!!"     );`
напиши `alert("error!!");`
you don't need to call declareDbProfile....
Considering that this is C++, probably you can create a class, that'll embed most of this structures to share all this code? It's really almost equal.
And a question: if .travis.yml is in the .gitignore file, how come its still being tracked?
You never create the initial ref anywhere but you ref and unref it.
wow... break on null, really?
Same comment as previous loop.
как мне запретить generated файлам здесь показываться?
I mean, "an" suggests "one" to me. I'd put "followed by optional whitespace", what do you think?
I think we can just do:

``` ruby
options[:path] = args.first.to_s if args.any?
```

I don't see any situation where it's going cause a problem. In fact I'm wondering whether we should just use `args.flatten.join('/') if args.any?` that way you can do the following:

``` ruby
scope :api, :v1 do
  resources :products
end
```

wdyt?
I think that the indentation here is wrong
gah, yep :(
I rebased and updated.
bro, this tap does not save a single line
The default for any Int is available processors?

I have alternative property passing code that looks more like:

```scala

def parseInt(x; String) = for {
   value <- Option(x) 
   number <- util.control.Exception.catching(classOf[NumberFormatException) opt value.toInt
} yield number
def getInt(name: String) = parseInt(System.getProperty(name)) orElse parseInt(System.getenv(name))
def getIntOr(name: String, default: => Int) = getIntProp(name) orElse default
```

You can, of course, clean that up a bit.  The core idea though is to also allow environment variables to be used.  It gives a little bet extra flexibility.

In any case, food for future thought.  I'd hate for us to string a bunch of code like this through the standard library if we can consolidate a bit more.

I'd be nice if we had a standard `SystemProperties` object somewhere where you could call things like getInt, getFloat, getString etc.
You use "rescueException isDefinedAt exception" below but "f.isDefinedAt(exception)" here
the call to backtrace will limit the result to fit the list. in the unlikely scenario where the stack is > 100 entries, we'll only see the top 100, which seems reasonable and useful enough for debugging, but I can change this if you want.
> Wont this loose the second and third expression here:
>
> `(u, u**2. (u,0,10))` for a parametric line
>
> `(u, u**2, u**3. (u,0,10))` for a 3d parametric line

Ahh...my shortsightedness. Yes, I'll fix that.
To avoid the python 3 problem this could be written ```tp2 = TensorProduct(JzKet(1,-S(1)/2), JzKet(1,-S(1)/2))```
sargs = list(self.args)
hit = False
for i, term in enumerate(sargs):
    if hasattr(term, attr):
        newterm = getattr(term, attr)(deep=deep, **hints)
        if newterm != sargs[i]:
            hit = True
            sargs[i] = newterm
if hit:
    return self.func(*sargs)
return self
```

This avoids creating a newlist; the way you have it, just having the attribute will require a rebuild even if no change was made.
Add a full stop at the end of this line, please, since it's not the only phrase of the docstring.
> I did not understand the "XXX" message. Shouldn't you add the failing example as decorated test?
>
I don't have a failing example anymore.
It would have been good to have another set of accustomed eyes (not mine) look this over before merging.
Why is contravariant vs is covariant?
I don't think you should make this change, Mul does some handling of Rationals that Pow shouldn't deal with, given the args structure of a Mul. ``ask(Q.integer(Rational(3,2)**x), Q.even(x))`` will go into an endless recursion with this, when in master it properly gives False. If you can think of a case that is mishandled, you can create a Pow method.
@resurtm can you please check kk translations according to number choice formats?
IMO this is not needed... the documentation is clear $this->option should be an array it cannot be null or false or empty string it can only be an [empty ] array, so it's better to check for that
Indentation here should be 4 spaces from the `if`.
In fact, what I want to do is to add a Bootstrapper if needed so that the user can define a simple class that calls .init() and that he can inject whatever he wants in the initial loading phase of the app.
Do we want resetConnection the underlying connection? Previously the connection itself was never exposed, right? Maybe we should make this void?
Sure, if answer is NOT present then ProtocolError is raises inside DataBuffer.
The driver must make sure that status and answer is always present. Perhaps it needs to be documented.
That was the idea when i did the routing module, I can still create a custom exception in httprpc but i think rule is difficult to move at this occasion
Fixed. Also, just found out I have commit rights on this repo...
Are the other exceptions, `socket.error`, `httplib.HTTPException` important to test for too?
Mach dir mal über die Übersetzung keine Sorge. Prinzipiell sollte es reichen, die .po und .mo Files zu aktualisieren. Allerdings ist es schon eine Weile her, seit dem wir die übersetzung das letzte Mal angeschaut haben. Es wird ein eigenes Arbeitspaket sein, zu überprüfen, ob gettext die bevorzugte Lösung ist oder ob wir was anderes einsetzen wollen.
This conditional is never false.

```js
var change = DS.OneToManyChange.create({
  // ...
});
```
I don't want to add support for scopes in Pimple... also because it is so easy to work around this issue.
If `index.php` returns false the file is handled by the webserver
please remove these extra blank lines, only one is needed.
For better or worse the current implementation always does chunked for request_block, but can we assume that will also always be the case here?  ie I don't believe S3 supports this.
Code style: don't put extra whitespace around parentheses. Only exceptions are keywords followed by a parenthesis: "if (", "for (", etc.
Why does the analyzer complain about this? Passing in NULL is a programmer bug, an empty string is not technically an error (albeit pointless).
This is problematic because if I want a a string with value '4' it is a valid string.
AFAICT this is the only use of retrieveAggregates/4, for which I suggested to replace the start and end parameters with an Interval.
I was under the impression that the only callers to this method would be internal, i.e. it gets called within the scope of `removeDomain` and `removePair`. The intention of this is cleanup is to prevent any dangling checkpoints arising.
CS: no newline for open curly brace
can we rename this to "findWorkerForTask"?  "get" is throwing me off...
"Remember me" login key expiration length (in days)

Or, Remembered login key expiration length (in days)
The only cache driver that uses a  cache directory is the file driver. Therefore this function does not belong in base driver.
Why are these permissions named after a "wiki"? This is a post revision feature not a wiki.
Still no suggestions?
Docblocks please.
why not just `(options[:sudo] || true)` or `options.fetch(:sudo, true)`?
Perhaps yes, or better yet read() should call readNoMerge() and then merge the results.  I will look into this.
yes, still need official confirmation but i'm reasonably confident without it.
will chase this up. suggest a new issue is raised, if you agree?
Both occurrences of `==` should be `is`.
raising an error *after* creating all the devices etc. makes no sense
for ip in self.ipaddress_set.order_by('-last_seen'):
    return ip.last_seen
Could you get rid of one of those empty lines? You know, me and whitespace... :)
Sure, that looks a bit cleaner. Thanks!
Style nits ;)

Please add whitespace padding to the inside of the invocation parens on each of these, thanks!
This is almost unreadable.
Reasons to avoid the name "blob"
- could be confusing this with Postgres "Large Objects"
- "blob" is a reserved keyword in the SQL Standard

I suggest something more like binarystream, bytestream, bytea8, or something similar.
Oh wow! Mind = blown. Homebrew is smarter than I thought.
Obviously I can't read; I assumed this was operating on the Makefile.
alias_action :delete, :deprecated => true
Oh, worth mentioning, it would also be semantically correct to use defaultExecutionContext here, but since the internal executor is only 8 lines of code I figured why not avoid the overhead; there's no reason to dispatch to a thread pool when we know the app executor will dispatch to the app anyway. We know this internal executor is always just an intermediary.
I think I'd be +1 on allowing the client to override this by specifying it in query string of the URL: `smssync/<account_id>?country_code=27`. Thoughts?
there seems to be a consistent way of printing a period (.) at the end of each line. this one is missing it.
typo.. "-Object"
I guess because we are handling blocked contacts the "easy" way and not the "correct" way :) Okay let's leave it without message_ack for now.
Done, added test
Shouldn't that be `> 1`?
@dsaff 
pls see my latest commit (removed license in POM)
These changes are related to your other pull request.; please remove it. It looks like you made the change based on the code in your previous pull request (a246413bbc5732ae66bdb0f35d43ca329efd9c24). You can simply cherry pick the last commit into another branch.
I was not accurate enough. Fixed it today.
This method is never called, is it? If not, let's please remove it.
Probably don't need the booleanization, the boolean operators will do the same implict conversion.
If you wrap it like `[WARN]`, we won't get false positives if the string `warning` appears elsewhere. I agree with @dizzyd that we may as well record `[:level]`.
What happened to the typeof friends check?
I split this into #1604. I did not assign it to anyone and wasn't sure of the priority. This may be a good starter bug.
I agree. We should allow the user to get back to the original color. Instead of keeping initialColorString and comparing with it here, I'm removing it and comparing to this.color right here . This way we're not doing redundant work if the user selects the same color swatch again, but still allowing the user to set any color.
Not to nitpick temporary UI too much, but it we might want to experiment with right-alignment -- I think it could work even better.
verify exception is thrown
Actually, I prefer the other solution. But I think it is acceptable to show encoded url in the list.
Very good point, I agree!
This FIXME should be shorter.

Testing the ability of commenting on the diff in a pull request...
I stub'd a method to zero, making it irrelevant.
As you can see `value` isn't used in the `if` block, but it is used in the `else` block. However, the `else` block uses the `foo = bar() until foo` trick, which implicitly defines the local `foo` before hitting the `until` loop.
You don't need to load the whole file into memory onto the response - even the file can be sent back as a stream:
http://stackoverflow.com/a/8601118/52302
Is name a human-readable field (unicode), a slug field, or something that's limited to a small set of choices?
Newlines inbetween elements like these
Moved to DefaultEvent
So leave it for now like this. Just add a TODO comment that we should check if the callback is really called as a consequence of the user pressed Facebook login button before.
Is `p` necessary now?
Please add an explanatory comment to _internal and _external, and how they are used.  In general, this code needs a bit more documentation.
Yeah... I noticed that. Don't know why I did.
you don't need to do div > .butter-unit, just do  .butter-unit. No need to add indentation
Move this comment inside the else block, and indent
This will turn it into a String vs. a Number, which seems wrong to me.
Why do we need to call this in a setTimeout?
this `<a/> is pretty lonely all by itself. Lets put it back
I have a serious problem on my test case. I will redo my tests today
This was a mistake, it should have been left in.
For better output, use DateTime instead of Date. This way, you can also render the time of a post - useful if you're posting more than once a day.
Please don't break the like here, screens are wide enough nowadays :-)
@sam Clear All Data in the settings menu will also do it, and much easier for the user.
let's default to false to begin with
Ok. I'll go ahead and do that then.
maybe closing "$ARGS" to "", just to be on safe side when one of the values has spaces
Why not use `Properties.load`?
Dude, I'm just trying to help.  I know puppies aren't free, but do you really need a separate carrier for each puppy?
建议改名为checkout
weirdly, I don't see these ids appear in Firebug...
Not your fault but can you move the '{' on the previous line to stay consistent
We could probably drop the Settings specific end of this file.
The app now needs to be added to
`apps/webapps.json`
Sorry I just thought about that. But If an application called preventDefault before the homescreen on one of those events you will never receive them.
And so the setInterval will be impossible to stop. Can you ensure setInterval will be cleared once you reach the minimun/maximum level? You probably needs two differents timer variable too.
What about |EventUtils.swipe = ... |?  I think it will look looks like its part of the EventUtils framework.
Don't all Python functions return ``None`` by default? I was trying to just write a lambda function that did nothing, and ``lambda: pass`` doesn't work. I guess I could write out the function and use pass - up to you :-)
@eteq: what @andycasey said: For now only the same masks add up. In the future one could imagine that it checks if there's a converter available before throwing an exception.
When coordinates supports arrays this should use np.sqrt not exponentiation:
```python
In [4]: a = np.arange(1000)

In [5]: timeit a ** 0.5
10000 loops, best of 3: 31.4 us per loop

In [6]: timeit np.sqrt(a)
100000 loops, best of 3: 13.4 us per loop
```
Done in 3c179d1.
i believe the reason the queue is in the payload exists for retrying the job from within resweb. See: https://github.com/Pyres/resweb/blob/master/resweb/core.py#L83

There might be another way to handle that from resweb, so let me think about it.
In any proper OS install, the OS is responsible for the version of OpenSSL used.
Yay, fewer places using pass by reference is a good thing.
I kind of prefer `if (xor(is.null(branch), is.null(pull)) stop(...)`
Also, a quick Google implies this function's name might be `sceKernelLoadModuleDNAS()`, but not sure if it's correct.

-[Unknown]
I had also thought of just calling it `ESC_MAGIC2` to keep it shorter and as a mnemonic that we're really making it be always the first twice. Up to you...
in general, please increase the quality of this class, similar to BatchError in sqs, which has javadoc, builders, etc.
lets not override just to set the timeout.  we have enough code to maintain :) please revert this and add the corresponding timeout property in the ApiMetadata class. which is a one-line change.
It's not true that DefaultWeightedEdge is the only provider.  It's just the default.  Any class can be used for a weighted edge; it is up to the graph implementation to decide how to determine the weight.
you might want to rewrite it on a single line to improve the readability 
buffer.append("+(").append.(q).append(")")
This is cool, nice job getting the CSV parser hooked up!
Both are correct, unless this syntax has changed in 5.5.
This one line is the only line that changed in this file between this pull request and the other one.
what about `signal_stop()`? Both a verb and uses `stop` instead of `quit`
Time measurement is already taken in ExecutionResult
Can you rephrase that last question? I don't understand what you're looking for.
Right... random categories should be removed... it was from a test installation ... will amend..
Actually shouldn't this be `('sage: ', '....: ', '>>> ', '... ')`? See also [trac #10458](http://trac.sagemath.org/sage_trac/ticket/10458) and [trac #12415](http://trac.sagemath.org/sage_trac/ticket/12415).

But I suppose changing it doesn't necessarily belong in this pull request...
Зроблено https://github.com/fre5h/fwdays/commit/2200d0c2d0d3d74132ca901c72cf31c9c7d0604b
typo: Once read
`ip` should be uppercased.
Because it's a constant value. Actually this should be a `const`, but PHP doesn't support that for arrays.
This is wrong as you modified ``$name`` several times. You shuld keep ``$key`` to store the original key.
Do you have a specific example to show?
The @api tags must be removed.
Alright, thank you for the explanation!  I didn't really pay attention to how it happened in Google's product.
This section appears to be a duplicate of the previous.
Thanks for the review. I've incorporated your suggestions. :) Shall I merge the fix now?
Sure, I can back out the workflowId parsing stuff for this review. I'll
post an update shortly.

On Fri, Oct 5, 2012 at 12:41 AM, Bill Graham <notifications@github.com>wrote:

> In common/src/main/resources/web/js/ambrose-ui.js:
>
> >        } else {
> >          _dagUrl = 'dag';
> >          _eventsUrl = 'events';
> >        }
> >
> > +      _dagUrl = new URI(_dagUrl);
> > +      _eventsUrl = new URI(_eventsUrl);
> > +
> > +      _workflowId = params.workflowId;
> > +      if (_workflowId) {
> > +        _dagUrl.search({ workflowId: _workflowId });
> > +        _eventsUrl.search({ workflowId: _workflowId });
> > +
> > +        // parse workflow id for component values
> > +        var components = _workflowId.split(/!/);
>
> Can we merge all but the _workflowId parsing part? It would be good to
> support passing workflowIds to the API and avoid forking on the index.html
> filename change.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/twitter/ambrose/pull/44/files#r1769604>.
>
>
Not sure if job selection has side effects I wanted here. Feel free to remove if UI seems stable without it.
I didn't declare this, copy paste from header to make them private. will remove.
alan khoobe dastehs nazan :D
save нуэно вызывать автоматически при любом измеении current_date и pixels_per_day через евенты:
```
setmodel.set(attributes, [options]) 
Set a hash of attributes (one or many) on the model. If any of the attributes change the models state, a "change" event will be triggered, unless {silent: true} is passed as an option. Change events for specific attributes are also triggered, and you can bind to those as well, for example: change:title, and change:content. You may also pass individual keys and values.
```
Hmm, I think there are reasons why people would want access to the split view in their drawing block. ;)
Yes you are right I add it thank,
I'm fine with the decompyle happening here in concept, but I worry that the operation might be very slow for some files...  Have you done any performance analysis?  If it can be a slow operation, we want to give the user some way to "opt out" of decompyling during extraction, since they can also decompyle later by just opening the pyc file.  We should also add "YesToAll" and "NoToAll" options for the decompylePyc, in case they already have .py files and don't want to confirm all 200+ files one at a time :P
Ignore it.  I forgot to exclude this file from commit.
These calls take a variable called "options" in pulp. So 'options' was added so it gets included in the local variables required section.
Ah.. I suspect it's just a side-effect of when you want to execute the `else` block.
Just return self._geoms?
This comment could be squeezed onto one line now.
Hey Cory,

Yes, because it goes right above search. But I'll think more about if that makes sense.

Best,
Daniel

Daniel Cothran
Program Officer
John Snow, Inc.
AIDSTAR-One
1616 Ft Meyer Drive
11th Floor
Arlington, VA 22209
(tel) 703-310-5123
(fax) 703-528-7480
dcothran@jsi.com
www.jsi.com
www.aidstar-one.com 
>>> Cory Zue <notifications@github.com> 10/16/12 18:01 PM >>>
> @@ -6,6 +6,7 @@
>  
>  <div class="module">
>  	<h2>All HSAs</h2>
> +	<p>Click on a row for more information, or</p>

did you mean to have ", or" here?

---
Reply to this email directly or view it on GitHub:
https://github.com/dimagi/logistics/pull/26/files#r1860740
which one? (just for reference)
I think we dropped that plan. Would only add complexity IMHO. I prefer it in the CMS, it's still easy enough to disable.
Its a plain, no its a train no its Captain Obvious!
That's a job for Remy. ;-)
oh right, this line needs to be replaced with

    			this.element.children().not( this.panels ).each(function() {
    				maxHeight -= $( this ).outerHeight( true );
    			});
For sync with the rest of the code, I think this should be #FFFFFF
You dont need `!!` here
Is the BB structure and any `<a>` needs an `href` for avoiding un-wanted behaviours
vibraction -> vibration
@alivedise told me that he handle the HOME key by have it doing nothing :) in parity with other phone OSes.

We could confirm with UX on these detail behavior later.
Someone changed this font size in grid.css but not here
why 100?
Let's put a clearer comment ("once we have gecko working properly is rather vague" ;)) like:
```
// XXX: Hack hiding the message class in the icon URL
// Should use the tag element of the notification once the final spec lands
// See: https://bugzilla.mozilla.org/show_bug.cgi?id=782211
```
no problem, we'll leave it to a separate localization pull request.
If the week is stored as bits then you can use the actual bits from a number and use bitwise operator to get the info :)

(do this in a follow up)
@alexch Do we have other dependencies that won't *install*/*build* on Windows? (rerun? I know it's not suppose to "work" on windows, but is it going to blow up `bundle install`?) From the sound of it, it sounds like @intuedge  couldn't even finish `bundle install`, so it's not too surprising that `rake`/`rake spec` didn't work for him.
~~~
そのジョブに関連するフィルタされたヒストリが返ってきます。
~~~

→

~~~
そのジョブに関連するフィルタリングされた実行履歴が返ってきます。
~~~
原文には `` がない
I believe my latest changes to edts_code also fix this problem.
I changed them all to my other email. I can discuss in person why.
I could remove it. I left it in so it would be more obvious that it can be re-added.

Actually I will remove it, and add some comments on this, which I wrote this morning:

    https://github.com/IoraHealth/IoraHealth/wiki/Boxes
I'll check this one.
Creo que con esto bastaría https://github.com/MilkZoft/codejobs/blob/master/www/requirements.php#L12
Why do you want to check in the pyc files here?
Fixed. I deleted the action parameter on the forms.
Man. I wish I would have remembered this. I spent some time yesterday
tweaking a way for easily setting up your environment, but autoenv is a
much better solution than  the one I came up with.

-Wes

On Wed, Aug 8, 2012 at 7:40 PM, Nick McLaughlin <notifications@github.com>wrote:

> In .gitignore:
>
> > @@ -1,4 +1,5 @@
> >  venv
> > +*.env
>
> For convenience, I used a script called autoenv<https://github.com/kennethreitz/autoenv>that automatically executes files with the .env extension in a directory
> when you cd into it. This allowed me to put all of my export commands for
> exporting the various private constants needed for GitZen into an .env file
> that would be automatically executed when I entered the project directory
> for GitZen. I want to ignore that file because it contains private
> constants that should not be publicly available on GitHub.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/PolicyStat/GitZen/pull/25/files#r1338073>.
>
>



-- 
Wes Winham
m. 405.206.4913
w. 317.844.6885 x117
Instead of assigning a string value to what we would expect to be a list, create a boolean that holds that value. More clear.
This should have the same effect as line 158. Why is this needed here?
This IE-specific code in Brackets was somewhat vestigial from our first attempt at the scrolling stuff. After more testing it turns out we only need to do this for IE7, and the width was smaller than originally thought.
That looks better, but my inclination would be to do pass through where we can and then extra methods. That'll have the advantage of making it clearer where we're doing extra work (as we may well want to subsequently clean up after ourselves and standardise naming)
Not necessarily required.
This is a "just" terminology, but this isn't a poll.  Polling means repeatedly checking something, on a fixed schedule like once a second or once an hour.  Instead, this method starts the websocket, so a name like `start` or `startWebsocket` or the like would be more appropriate.
This shouldn't be writing anything back to the config.  Better to store this computed data in an object attribute somewhere else.
Changing an `upgradeToVersionX` function after it's been in released code is a bad idea -- if this happens, then we'll have version-3 sourcestamps both with and without sourcestampsetid.  If it's important to remove this attribute, best to do so by adding a new version.  If it's not important to remove it, then we can just ignore it.
if only user= then add

(?:\s+user=.*)?\s*$

may be ;-) ?

better to have them anchored because then I believe someone could spoof
ip with a user=rhost=1.2.3.4

and since .* is greedy it would get there to match

so please verify that it would work with anchoring as desired as well
for the case of spoofing

On Tue, 11 Dec 2012, hamilton5 wrote:

>    In config/filter.d/dovecot.conf:

>  > @@ -15,6 +15,7 @@
>  >  # Values:  TEXT

>  >  failregex = .*(?:pop3-login|imap-login):.*(?:Authentication failure|Aborted login \(auth failed|Aborted login \(tried to use disabled|Disconnected \(auth failed).*rip=(?P<host>\S*),.*
>  > +            pam.*dovecot.*(?:authentication failure).*rhost=<HOST>

>    no sometimes there is a user=name at the end..
>    [1]…
>    On 12/11/2012 9:46 AM, Yaroslav Halchenko wrote: In
>    config/filter.d/dovecot.conf: > @@ -15,6 +15,7 @@ > # Values: TEXT > # >
>    failregex = .*(?:pop3-login|imap-login):.*(?:Authentication
>    failure|Aborted login \(auth failed|Aborted login \(tried to use
>    disabled|Disconnected \(auth failed).*rip=(?P<host>\S*),.* > +
>    pam.*dovecot.*(?:authentication failure).*rhost=<HOST> I believe this one
>    could be anchored at the end with \s$, right? — Reply to this email
>    directly or view it on GitHub
>    <[2]https://github.com/fail2ban/fail2ban/pull/94/files#r2378678>.

>    —
>    Reply to this email directly or [3]view it on GitHub.

> References

>    Visible links
>    1. file:///tmp/mutt.html#
>    2. https://github.com/fail2ban/fail2ban/pull/94/files#r2378678
>    3. https://github.com/fail2ban/fail2ban/pull/94/files#r2379682

-- 
Yaroslav O. Halchenko
Postdoctoral Fellow,   Department of Psychological and Brain Sciences
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834                       Fax: +1 (603) 646-1419
WWW:   http://www.linkedin.com/in/yarik
You have to deal with err before that
can we make a `unit_membrane.sh` with corresponding `components/tests/unit_mem*`?  If we add a significant function, we should have a systematic way to test it.
maybe we can do a stack alloc first, only heap alloc if condition below is true, for perf reasons
Please append these to the end of the enum so that no existing codes are changed.
is there a reason for the 128 here?
How can I find out what's in this diff (and the other subproject changes)?
Ah, I see, this makes sense.
We need the icon position for cpanel modules on the right
and other actions?
Currently established convention for module names is `hypen-delimited-name.js` let's stick to it please.
got it cc32520.
Would it make sense to set the origin in user.js to use the originHREF instead of having a separate field?
Simplified - thanks, @zaach
hrm, bypassing the check for unauthenticated cookies changes the behavior of stalled-db situations, causing one of the stalled-mysql-test.js tests to fail.. looking into that now
This should be wrapped in a try/except to catch `DatasetNotFoundError` and turn it into an `Http404`, otherwise we end up sending 500 server error back to the clients. (Our existing code doesn't do this yet, but if we're introducing new code we should make sure it does).
The number of characters in `countPendingAddonsAndNotifyToRestartCallback` function name is [too damn high][1].


[1]: http://mozillamemes.tumblr.com/tagged/too-damn-high
@dbussink I think here ```$,``` is restored [click](https://github.com/LTe/rubinius/blob/c16e882a5b1851eaba663fd9cb1cbe79dbe5a8f5/spec/ruby/core/array/shared/join.rb#L10)
I think the default implementation of ``this.updater`` should be updated to return ``this.itemLabel(item)``, so that it does not break when people start passing objects (they will still need to update the method if they don't want to use the label as selected value of course). what do you think about it ?
I have it is an a normal dependcy because we use the template generation...


On 26 November 2012 16:06, Sindre Sorhus <notifications@github.com> wrote:

> In test/test-api-v2.js:
>
> > +  before(generators.test.before(path.join(__dirname, 'temp')));
> > +
> > +  // generators is an instance of event emitter.
> > +  it('generators is an instance of EventEmitter', function() {
> > +    assert.ok(generators instanceof events.EventEmitter, 'Not an instance of EventEmitter');
> > +  });
> > +
> > +  it('generators.Base is the Base generator class', function() {
> > +    assert.equal(generators.Base.prototype.__proto__.constructor, events.EventEmitter, 'Not an EventEmitter');
> > +  });
> > +
> > +  it('generators.NamedBase is inheriting from Base generator class', function() {
> > +    assert.equal(generators.NamedBase.prototype.__proto__.constructor, generators.Base, 'Not a Base class');
> > +  });
> > +
> > +  it('can be hooked using a grunt instance, from which we read cli tasks and options', function() {
>
> @PaulKinlan <https://github.com/PaulKinlan> That's not a problem. Just
> add grunt as a devDependency, and use it as much as you want in testing ;)
>
> What I'm talking about here is to remove grunt as an actual dependency.
>
> Sidenote: grunt 0.5 is going to be more split up, so you'll be able to use
> the various components individually.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/yeoman/generators/pull/141/files#r2223576>.
>
>
Era nella precedente versione del manuale... E' da rimuovere.
Since class names in strings are always considered fully qualified, the leading \ is not necessary. (I'll likely make that change during merge.)
Oh noes, extra line!
During development we encountered that function elsewhere and it looked like something we ought to use, but we aren't entirely sure if it's the right thing to do.  Some advice would be good.  If you aren't sure either, I'd say the comment is spurious (as the tests pass).
Hmm, I'd rather ditch the helper and just do
return self._attr.get('altitude', [])
Everything but the call to encode exception.  No reason to pollute the stack.
These environment variables are set by the settings include file automatically when drush commands run, though they only run when drush bootstraps to the point that it loads settings.php. However, cache-clear must bootstrap that far, because it needs to connect to the database. So I don't think these are necessary.

Also, the AH_SITE_NAME setting is wrong. It isn't $site.$env. It's actually $site$env, unless $env is prod, in which case it is $site. But that also might change in the future. Customers should not use AH_SITE_NAME in their code. So we definitely do not want to set it (incorrectly) here.
This looks a little dirty? Any better way to do it?

You know you can override the `base.css` in `app.css` if needed.
Same thing here, should be !== instead of !=.
please instantiate them inside the data provider then
yeah. another alternative would be to use bit masks to enable documents to have multiple "states"
It is. Some time ago we talked about that on IRC, but I have not yet found the time to prepare a PyPi package.
the variable should be caml-cased
... but this needs a foreign key.

Don't be afraid of doing a `rake db:rollback` if your migration is not to your liking. Then you can edit the migration and rerun it.
Please make this `call_process` (to follow python coding conventions).
Change "Search popular groups" to "Search Groups with Mozilla Stewards"
In theory. There's actually no styling applied to them. Having classes at all might be a little forward-thinking/pre-optimization.
See comment below, this shouldn't be moved
can't we use input[name='products'] as a locator?

then we will just have on line: #152 
<code>self.selenium.find_elements(*self._article_topic_locator)[index].click()</code>
Newline required here.
Oh, and you don't really need .try here - you're calling the methods on self, so you know it exists. try is most useful in situations where you're calling a method on an object that may or may not be nil.
remove "+ 1" from max :/
same here, also I'd ditch the comment personally since the string describes it already. Maybe change the response to "invalid json, empty body"
Why is it necessarily inside the images folder when we're debugging?
why not run it with bundle exec instead? was that too slow?
nit: Seems like we can save a few bytes by returning `options.data` from this call.
Given that we might want to run this code also if the master is on windows, you should also copy the artifacts from the tools job.
done (and this looks way way more convincing now)
A single comment.
the reason we can't do this (at least only this), is because you could for example install say a `component-server` command, and `component server` would still work, much like extending `git`. We could however maybe do a quick `stat()` of the join, if that exists use that, otherwise PATH like before
Couldn't you simply rely on the autoloader generated by Composer ?
please fix it as the line is already part of the diff anyway as you removed the semicolon in it
In the Fridge CS, we add brackets to each comparisons. So your CS is good on this point. But, If the condition is not a comparison, simply don't use them. 

Your condition should look like:

```
if (!is_resource($value) && ($value !== null))
```
зачем нужен индекс? надо научится брать последний элемент вектора и удалять его
Here I think is the approach that we should follow. The idea is that we
cannot know for sure if the entity that was saved is exactly the same as
the entity that is stored in DB - take in consideration after insert/update
triggers. Those can change the values of the entity stored in DB. For this
reason I loaded the entity from DB and return what I know for sure that is
stored in DB.

On Wed, Sep 26, 2012 at 12:24 PM, Richard Rodger
<notifications@github.com>wrote:

> In lib/plugin/mysql-store.js:
>
> > +      qstr.push('INSERT INTO ')
> > +      qstr.push(getTableName(canon))
> > +      qstr.push(' SET ?')
> > +
> > +      var query = qstr.join('')
> > +
> > +      self.connection.query( query, ent, function( err, result ) {
> > +        if ( err ){
> > +          seneca.fail({code:'save/insert',store:self.name,error:err},cb)
> > +          cb( err, ent )
> > +        }
> > +        else {
> > +          seneca.log( args.tag$,'save/insert',ent )
> > +          // load from db to make sure is ok
> > +          ent.load$({}, function(err, result){
> > +            if ( err ){
>
> why do we need to do load again? mongo-store just sets the new id and
> returns the input ent
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nodejsdublin/seneca/pull/4/files#r1693352>.
>
>



-- 

*malex*
Maybe even `(= (count %) (count obj))`. Something that reflects that the two objects are similar.
We'll still need the additional fields for this that allow for specification of the appid, etc.
Yes it helps the UI determining the execution context.
Again, validation can move into a 

it('verify threaded comments validation)

function
Why wouldn't we just use the language features that best express our intention? `for (var i = 0....)` if we need to use an index, `forEach` if we just need to iterate the elements? Easier to read, IMO.
We don't want to distribute it. But there is a bug in  the version provided by the JDT which makes it impossible to use with the Scala Debugger. I'll push a patch upstream.
probably make sure this is space delimited, so add a trailing space to "boot". also, a better way to construct this is `text.indexOf("houndbot boot ") == 0`
don't wrap the code in ``else`` here as the ``if`` returns. It is unnecessary
Maybe num_users should just be precalculated in the initial page load with a `User.objects.all().count()` or something -- which presumably would be a fast query no matter how many users there are.  That way the user would immediately see how many users there will eventually be on the page?
The PHP version requirement was changed to 5.2.4, so is this docblock changed in each file.
И опять таки, переменная DNALength не изменится.
Не вижу проверки на случаи, когда percent>100 или меньше нуля. Да и вообще, percent - это проценты, а то, что у нас 100 символов в последовательности - счастливая случайность.
А, вижу, вы в курсе. =)
"Создайте имплементацию (реализацию) метода mutate. Метод должен заменить X процентов символов в массиве DNA в случайном порядке; Х – значение переданной переменной типа int. Нужно заменить строго Х процентов, то есть заменять одну ячейку можно максимум один раз." - что в этих словах не понятно?