labofsahil/github-event-dataset-2012 huggingface dataset:
This change behaves unexpectedly for sufficiently large lengths:

```python
>>> x256 = ['x' for x in range(256)]
>>> x257 = ['x' for x in range(257)]
>>> len(x256) != len(x256)
False
>>> len(x256) is not len(x256)
False
>>> len(x257) != len(x257)
False
>>> len(x257) is not len(x257)
True
>>> 
```

```is```  compares the ids of objects, sufficiently small integer values have an invariable id value (which is likely an implementation detail and not to be relied on), but large enough integer values do not have an invariable id:

```python
>>> id(len(x257))
18775136
>>> id(len(x257))
18775088
>>> id(len(x257))
18775136
>>> id(len(x256))
18235008
>>> id(len(x256))
18235008
>>> id(len(x256))
18235008
```
use class="final_answer", and take out the color coding here.
Can we avoid using a curl here?
Did you mean ``@with_injector(UsernameModule)`` here? To test the new class-only modules?
You already inherit this from the parent - no need to repeat it here.
очень странный способ. разве этому классу нужны эти components
this is a native test.
It uses native events that are not available in MAC

@krupa  Should we keep it in our testrun?
really sorry for this one, my mistake :(
This method can be removed when the change mentioned above is implemented.
By this, I mean the `.cal-day`s and `hour hour-label`s. You can do `for i in 7` and `for i in 12` twice.
You can set it within your git configurations, but that's not enforceable.
If it's a hash, you can just do modules[i].config.options['twitter-authentication']
indentation and multi-line param
I might be missing something, but I don't see the 2 seconds timeout advertised in the previous comment.
also would be difficult to answer those questions without a refactor
i wanted to keep this spec identical to the existing spec for the time being. can we fix this after the name change?
I like "допустимые".
Or just remove the part where it says to click. Something like that. "Démarrer le Travis sur vos projets puis faites un déploiement sur GitHub."
This elseif is useless, because if the first `if` match, it will throw a exception and the method will be stopped.
1.5.0 isn't on NuGet? Latest is 1.4.0. I can build it after setting it back to 1.4.0.
That check does still exist above my diff. I only removed the redundant check. The fact that the Unit_test::set_test_items method feature does not work with the Unit_test::results method in the latest stable version of CI led me to this fix.
Putting 1 << 32 in a variable is solely for performance reasons and may be inlined. It saves about 0.15 seconds for every hundred thousand requests.
1. на j не накладываются никакие ограничения. Проверьте хотя бы на отрицательность и больше 100.   (оба варианта приведут к зацикливанию).
2. Хорошо бы еще j - переводить из процентов в кол-во. 
3. while(i<j)  даже без проверки на отрицательные значения  j застрахует вас от зацикливания.
4. при j = 100 сколько проходов будет по while?
А вы попробуйте. 
--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev
HAVE_RPI is detected automatically based on the VideoCore library existence in the configuration script.
Should be:
  NSString *reviewURL = [templateReviewURL stringByReplacingOccurrencesOfString:@"APP_ID" withString:[self appStoreAppID]];
since [self appStoreAppID] returns an NSString* , and not an int.
Then you don't need the string format, the code was written using the constant.
again, "shown in the XDR..." can be dropped.
please, Hayk already made this to be universal: will be (sam.repo for your case)
If two different parts of the app call queueOperation it could mean something else is wrong, so I do agree with the NSAssert here.
I don't like to have a property that leads to computation. Maybe we can have a real times attribute
var currentTry (I think looks better).
Do you think its important to show them that the data is stored in the SQL database that they created in line #80?   So them how to run sqlite3  and query the logged message?
Although it isn't your mistake, could you please make this correct? It should be:

    if (!$isDebug) {
I don't like this method, the recommend way to start a symfony project is:

    $ php composer.phar create-project symfony/framework-standard-edition 2.1.x path/to/project
you should remove this line as it is duplicated (it is also at the end of the file, where links should be)
“给定键值” => "每个给定键值"
给定这个词直接放在这里容易歧义。
“一个”一般不用翻出来，只要它是表示泛指，不是真的说数量是1
You can use `instance_read` for this. I think.
It could be still be changed to something a bit easier to maintain, such as `AffirmitiveButton` (Ok/Yes) and `NegativeButton` (No/Cancel)
That is utterly bizarre (not to mention counter-intuitive)! I stand corrected and bemused. :)
At first I also did like that, then I saw some repeating code like get index, or get view, max view ... so I decided to merge. Let me know what do you think best then I will refactor accordingly.
Probably because I tried something else before, and it got deleted in the mean time ;-) Want a patch for this ?...
Please replace the manual list creation by a simple
return Collections.singletonList(new File(Settings.getGpxImportDir()));
double quote -> singe quote?
the only reason being, so we don't have that extra step of iterating thru a list, spaces are not allowed in usernames anyway.
is this just wsgi pipeline goodness? No need for a return i assume...
why do you need the lock here?
It seems you only need to protect access to _servers.size()_, so you could be more fine-grained.
This is pretty old fashioned, why not submit a Callable?
Galder told me how this is preferred method. It is a testng class if you can believe. But getCause is not ISE here, lines got tangled a bit. I throw ISE if code executed beyond val.get(). val.get() should throw EE and here I make sure we do not have two EE in the chain.
Hm, since we're mixing Functions and Strings now, maybe `typeof options[key] === typeof defaultOptions[key]` ?
Can you add a `.. versionadded:: 0.6` directive to the block?
Please, check 32d738878247cee69bc420e456ec24ca0086b64b. And https://github.com/klaussilveira/gitlist/pull/192.
No, I guess we can keep the directory name, it's generic enough.
Yeah I wondered about that… but it seemed odd to have a `void` returning method take a variable "by ref".
Try uncommenting the ```set_?lim``` calls and the data is not put in the correct range.
```
import matplotlib.pyplot as plt
import numpy as np


import matplotlib.transforms as mtrans


class div(mtrans.Transform):
    input_dims = 2
    output_dims = 2
    has_inverse = False
    
    def transform_non_affine(self, points):
        new_points = points.copy()
        new_points[:, 1] = 1/new_points[:, 1]
        return new_points
    
ax = plt.axes()
npts = 21
xs = np.linspace(-5, 10, npts)
ys = np.linspace(0, 5, npts)
xs, ys = np.meshgrid(xs, ys)
data = xs**2 + ys**2

plt.contourf(xs, ys, data, transform=div() + ax.transData)

ax.set_xlim([-6, 11])
ax.set_ylim([0, 5])

plt.show()
```

Changing the transform to fix inf points will solve the issue:

```
    def transform_non_affine(self, points):
        new_points = points.copy()
        new_points[:, 1] = 1/new_points[:, 1]
        new_points[new_points == np.inf] = 10
        return new_points
```

This means, if inf/nan is ignored at the level that this PR fixes, then we will get the "correct" limits.
+1 for using mozqa.com, however I would avoid using the index. For now it's fine but perhaps we should use something from http://mozqa.com/data/firefox/. We commonly use the pages in http://mozqa.com/data/firefox/layout/ for Mozmill FWIW.
Can we have this all on one line, I think it makes it more readable. Maybe we should use the same style as Web QA here (PEP8 except for line length). Not a blocker for me though. ;)
Похоже тоже дифф не с тем состоянием.
We'll revisit spec validation issues post RC1.
The spec allows for objects that implement __toString() - other stuff should go in context
Thanks. Fixed in aac1a7b.
I need to get this working properly
Rather than including a "for example" in the text, just make that a test and comment on it.
My text editor (sublimetext) truncates whitespace on save, hence the many diffs
Why did you remove the arrow?
This won't be found. The puzzle border should probably be moved to /ui/img
@bp323 single quotes.
This tells Sling to bind any AuthorizableIndexingWorkers that are exposed by other bundles. The bind... and unbind... methods do all the work of maintaining the list of AuthorizableIndexingWorkers.
No chance to know this, here too.
Then actually part of the change makes sense anyway, to display something like in EditAllBody if app list string is empty:

{if $APP_LIST.moduleList[$CATEGORY_NAME]=='Users'}
   {$MOD.LBL_USER_NAME_FOR_ROLE}
{elseif !empty($APP_LIST.moduleList[$CATEGORY_NAME])}
   {$APP_LIST.moduleList[$CATEGORY_NAME]}
{else}
   {$CATEGORY_NAME}
{/if}
How about moving this into a separate template file or at least using a heredoc?
Boo trailing whitespace!
Why this change? Each element *is* passed to the iterator as the second argument:

    ret[ret.length] = fn.call($, i, el);
Unless there's a supported, intuitive use case for passing a null key, that just makes the code longer and slower.
To avoid the costly `=`:
~~~clojure
(empty? (filter (complement pred) a-seq))
~~~
looks like a typo in the comment. You meant to say "in whyrun mode we do not want to keep the temp file", yes?
Yes you have to run both, but in previous comment you said to delete the other one(I might have understood you wrong).
I've created this signal and it works. But I'm not 100% sure about the code, the django docs were a bit confusing to me. Could you check please?
I don't like that you are duplicating basic map configuration here. Can you simply reset it to `myOptions`? (Which really have a bad name, please.)
Why `!important`? You should not be needing to use this?
See the comment about the script tags above.  Same applies here.
Can we change the defaults here so they are in line with what we shipped before? I don't want to break users site when they upgrade. So both ETDMS and OAI_DC will be MODS with different crosswalks?
Depending on how the UDP socket code is implemented, this could also be an indication of a network problem.  I believe this call will currently result in an empty dialog.
"alki" hardcoded?
Da könntest du jetzt die Methoden mit einer Map aus der Res Klasse brauchen
Unfortunately, that's a lot more complex.
The keystone is applied to an area anywhere in the image. The old system apply the keystone on the borders. Here, you define 4 points and the process transform them so each borders of this quadrilateral become horizontal (or vertical).
furthermore, points can be set almost as you want, so one may move just one point, for ex.
You can see, that the backtransform function is a lot more complex, not to speak about the transform one !
Arguments each on own line (see "method arguments" in https://github.com/pmjones/fig-standards/blob/63e6a4c6fe2eb1096d407a46394d253ac9241bd5/proposed/PSR-1.md )
Hmm, not following unfortunately. What element? What does it do?
This line looks strange anyway - why do you use summerizable.max() as count, summerizable.min() as sum, summarizable.sum() / summarizable.mean() as max and  summarizable.sum() as min?
this comment doesn't make sense, needs context/links or to not be here.
_zoomer_ele is set before this function is called, so as long as it's set to an element, this is safe. I can add a check that throws if the _zoomer_ele is not set correctly in set_zoomer_element() though.
Debugging code? comment.
At various points they're multiplied with `m_children.size()`, which is `size_t`. If they're just `int` then I get unsigned warnings. I could use `unsigned int` or `Uint32`, but I chose `size_t` because it meant the same type throughout, which seemed less ambiguous. But perhaps harder to read/follow sure, so I'm happy to change it something else.
This is not a good idea. `mocha` doesn't work with the last version of minitest
`displayUrl` is misspelled here.
Really?  Does everything still work with url encoding?  I know I was trying to remove this.
Turns out, this change causes a very subtle bug that I'm fixing:

psetToExs is a Lazy object (it's basically an abstracted query) that only gets evaluated at certain times.  Calling len() on it evaluates it.
B/C we removed the len(), this is actually just a query still.  So, when we index into it with psetToExs[n] two lines down, and try to save(), that does nothing.

When we have query sets, we should only iterate through them with this form:  
```
for i in qset:
```
Other wise we should evaluate them first with list(qset) and then do the loop.  But this means that fixing the list() namespace issue becomes more important, so I will do that today.
Does this work on the existing boto that is installed / patched?  Clearly this is better, so I don't want to take this out, just need to know what the install steps will be.
Should this line be commented out? It's not commented out in JBoss6SshSetup
Or at the very least deprecating it, saying it will be deleted before 0.4.0 is released.
Using repeater can cause things to seemingly pause for a long time for a user, so good to log the first (at least) at debug. We could/should then log subsequent iterations at trace and then the final success at debug.
ok, and have you been able to test this code path manually (presumably by going to the login URL when you're already logged in?)?
i expect use of   named.foo   in brooklyn.properties will become legacy as we introduce a better way, but we haven't yet :) so not deprecate/warn here yet.  updated comment to reflect this.
What about making the non-template functions into templates? Something like this:

    int indexOf () (String str, String ch)

Note the empty pair of parentheses.
How about "If the client code is deployed separately..."
I suppose I wanted to know if it was possible do it that way
The only purpose is to grab the upstream result. Calling `awaitE` immediately following `LastOne` will always give an upstream result, so the `const finish` expression should never actually be called. But there's nothing in the types to express this, so we need the hack to make it work. You can get rid of the whole business by changing the type to `GLConduit`, which will just force an upstream result type of `()`.

Yes, this is the kind of stuff I was worried about with the 0.5 migration. Fortunately, most people can likely just stick with `Conduit` itself and never worry about it. It's only packages where we care about supporting every possible use case that this kind of complication creeps in.
This is misspelled - it's "unknown".
I still need to make the documentation working. This directive will be still needed, but I desactivated it for now.
Shorthand is `$(function() {`, for what it's worth.
I do not think you need this in the image.spec.
Okay. I thought about doing that a while back, but we should discuss what's going to go where.
Why are we checking for a remote named origin? Why not check for any remote?
Accolade moet op volgende lijn ;)
Zie http://www.github.com/forkcms/coding-standards; of run tools/check_code
This logic should be extracted into a setValue method so that we don't duplicate it here and below.
Don't like my sloppy coding huh? :D
That's a typography link which doesn't have authority in the area of language correctness. You asked for ~~grammatical~~ orthographical correctness, not consistency. If this is about consistency, I'd rather see everything with small letters but I don't care, commit is coming.
Sure, we could pair on a refactor.  We could try using ```define_method``` instead and if that's not possible we can use instance variables instead of class variables.
I'm still nervous about using Time.now instead of Time.zone.now
Так есть параллельные обращения к toMerge на запись и на чтение?
further change it
This line is unnecessary, a fragment I missed before committing.
Looper.myLooper() == Looper.getMainLooper() should also do the trick I think
This is function that needs to be incorporated into the default command behavior of RHC::CLI after we merge more commands.  Until then it can stay here.
1. I think this used to be in the load_basic function but maybe you changed the code?
2. We don't really want a join b/c we'll be using a "preferential" approach to these tables. Try to load from the peoplesoft table first, if that doesn't work, use the old table.  Then maybe store where you got it from in the user object so that you knew for future reference.  Also, we could probably disallow saving to the old table.
Maybe rename the file to `california_housing.py` instead.
bad docstring indentation
I'll merge this but I think we should either remove this attribute or use a bunch of the blah blah HostEnvironment javadoc text that says it is meaningless.

My vote is "remove".
Changes look OK to me, but why the `::error` here and `error` below?
I like the idea here but i'm not sure we will have the time in our budget to make the changes right now.
Use the prefix underscore for these static functions, we'll reserve the suffix form for member variables only
is it possible to test for actual UI.View type?  Currently there is a gap between the test and the error message.

I put this comment in once, but it also applies to the other test that might throw the same exception.  The combination of both should be able to test that the passed object is actually a View.  Currently it looks like the any object would pass the tests without having to be a View.
Should autowiring be done considering that the delegate was configured as an instance? That instance may have been (should have been) initialized already.
should probably be implemented for consistency
I think that's how I would written it.
NB: bug fix
+1 on your idea.
This code seems a little weird, but it looks like it's not new--it just got moved... so probably harmless enough.  I'm just not sure I understand this comment: "We're using a per-thread cache of said module directories, so that we don't scan the filesystem each time we try to load something with this autoload instance"; if it's instance-level caching we're concerned about, why are we using a thread-local?
Not sure about this... Given that this is "chained" with another route,  shouldn't you always populate a `RouteMatch` using query params?
Isn't it usually always matching? (yes, this makes it difficult to use it to just build URIs).
Again, format issue
Removed the casting.
need to remove it
As I suspected, this breaks the fake (recorded) data feed, because the date range to search is part of the request URL, which is what VCR uses to match requests. A change in URL means the VCR cassette needs to change. In this case, the request in the VCR file is itself timeshifted, so a tweak to timeshift.rb is necessary.
switched both to maps. sounds good.
Same here, @unsymbol.
Can you add a test match where a team and it's alter ego both compete?
does amazon Linux current show up as osfamily == Redhat? if so, the differences should be within that block.
It's safer in that uncommitted, conflicting local changes won't block you from msfupdating forever. Remember, msfupdate is almost always about users and not developers, and it's usually run non-interactively.

Anyway, I've tested out git stash on my crazy minimal binary build so at least that'll save things. One sec.
0.upto(result.length/2-1) do |i|
What's the reason for using half of the max value as threshold? I would go with 0 as False and everything else as True - sounds much more intuitive to me.
The only time the vertex array for the north pole is undefined is the first call to `update`. Because the `boundingVolume` of a `Command` is initiallized to `undefined`, there will be only one instance created on the first call to `update`.
Part of the unusual way EventSource works is that there is no concept of done, that is, the server closing the connection is always and only considered to be an error by the underlying API.
All of this code was copied from `PerspectiveFrustum` and then generalized since it can be implemented in terms of an off-center frustum. This code has been around since the camera was started. Glad to have another set of eyes on it :)
Oh, yes.  Sorry.  Go on, I think this will be the last one before I merge all your changes.
Can you add Django cmdlets?
We can try to fix this, but it would require a large rework of the code base including the existing Management.CloudService project. Could we do this as a clean up item?
Added missing variable. This must have been added in master from another commit not back ported.
the minimum iOS version supported by Titanium is 4.0, so these osver fields could be omitted.
Should be:

```java
}
else
{
```
This is why I (still) prefer the original language for the handler ordering.  It avoids the pitfalls around "if fulfilled"/"when in the fulfilled state", etc.  Hmmm, although something like this may be ok:

>In the fulfilled state, respective `onFulfilled` callbacks must execute ...

The key is to avoid tying a handler order requirement to any *instant* of fulfillment/rejection, but rather making it simply mandate handler order, period.  In fact, imho, there needn't be any indication of when the handlers must be called, nor even the implication that they'll be called *at all* (both are covered by the existing 2.2.2.1 and 2.2.3.1)
Hi, I'm afraid `CGI.escape` breaks it completely. Here's what I tested and works:

    URI.encode(url_str, /[^-_.!~*'()a-zA-Z\d;\/?:@&=$,\[\]\]]/n)

It's basically what `UNSAFE` contains in Ruby 1.8.x with the plus sign `+` removed. The regexp starts with caret `^` so it allows the plus sign `+` to pass through unchanged. Can someone else test it too? @huhongbo?
As we're at it: move `xmlns` before `xmlns:xsi`
That's both a good point, and a good idea. I'll get to it then.
Mm I guess I'll remove dchar[] from the list
(Obviously this one wasn't necessary -- I just did it for consistency.)
I'd argue against it based on the fact that it could break code except for the fact that `isSafe` is pretty much completely broken right now anyway, and I tend to agree with your dislike for `isSafe` evaluating to `true` for `@trusted` as well. So, I'll make the changes.
done. oops :/
guess i was a bit too eager there. i went through it again to just annotate the specific methods (soundcloud/ActionBarSherlock@23d3cea1dbd3a213cbfdb6c5554499dae26c2b97). 

@SimonVT didn't find a way to exclude NewApi errors for certain packages, maybe this would be the preferable approach. On the other hand, having ABS checked by lint itself is not a bad idea either.

I guess the quick solution is just to exclude ABS from lint, since it's known to work on 7+.
again ``public`` keyword missing on methods in this class
why did you leave `ofColor c;` in, it doesn't seem to be used anymore?
lol. You should go ahead and get started on phpBB 4 so you can use those guidlines. If you get it started, it might not take so long to get it done when we get around to it.

Just kidding. :)
`<label>`
```
.table th,
.create-file p {
```
Spelling? Did you mean ```Surtyn```?
We might as well add `d3.v2.min.js` here. From the looks of it we're going to be using that library on every page that has a visualization (ie. all of them except for the home page).
This #define must be removed otherwise the library will always think it is in ATTiny mode.
Right, that got moved to the Note. I'll remove from the category.
A caveat with this change:  If there is no HTTPSHandler, then this variable goes undefined.  Then, when _get_data_server() is called, it will fail with an unknown "urllib_HTTPSHandler" when creating the ViewVCCachedServer.  At the very least, maybe a matplotlib.verbose.report()?
``attributes`` would be clearer than ``data`` - it's not clearly specified what it means to say that a certain object has something as "data"
а можно как-то задефайнить это для Qt 4.8?
Never mind - leave this as it is...
I really don't like this. You're replacing something that is unambiguous, with something extremely ambiguous. Pull request rejected because of this.
Note that the only time you would use this is on 32bit system, where you are checking if you're actually running on an x64.
ust to add my 2c - I think using a LiteralExpression internally (when configuring a value, or 'key' in this case) is very different from requiring a SpEL expression that returns a literal ("'foo'").

LiteralExpression is a very thin wrapper around a String. An expression that returns a literal is actually only a little more complex but, I agree that separate attributes is 'nicer'.

That said, the parsers are beginning to be littered with code like...

````java
boolean hasViewName = StringUtils.hasText(viewName);
boolean hasViewExpression = StringUtils.hasText(viewExpression);
if (hasViewName ? hasViewExpression : false) {
	parserContext.getReaderContext().error("Only one of 'view' or 'view-expression' is allowed", element);
}

if (hasViewName) {
    RootBeanDefinition expressionDef = new RootBeanDefinition(LiteralExpression.class);
    expressionDef.getConstructorArgumentValues().addGenericArgumentValue(viewName);
    builder.addPropertyValue("viewExpression", expressionDef);
}
else if (hasViewExpression) {
    RootBeanDefinition expressionDef = new RootBeanDefinition(ExpressionFactoryBean.class);
    expressionDef.getConstructorArgumentValues().addGenericArgumentValue(viewExpression);
    builder.addPropertyValue("viewExpression", expressionDef);
}
````
We should probably create a utility method and begin factoring these out.
This section is for outbound-channel-adapters parameter reference. So, i feel we should't be mentioning about the outbound gateways we will see in next section.
I thought "maybe it's a bug' referred to application code, not the language itself.

I agree that bugs in the language itself are extremely rare.

Sent from my iPhone

On 2012-05-06, at 4:50 AM, Danny Kopping<reply@reply.github.com> wrote:

>> @@ -14,7 +15,7 @@
>>     * Be sure to disable any caching technologies to ensure you're always looking at the most recent data, and that your _actual_ code is executing every time (not some cached version).
>>     * Have you tried turning it off & on again? =)
>>   5. **Blame your language.**
>> -    * Is the code you're debugging very complex or novel? Maybe it's a bug (highly unlikely)
>> +    * Is the code you're debugging very complex or novel? Maybe it's a bug (highly likely)
> 
> I'm curious to find out why you think a language bug is highly likely...
> In 5 years of working in PHP and ActionScript, I've never come across a language bug; but it's certainly possible that I just never happened to try anything that the language designers missed. Maybe we just say "somewhat likely" instead of "highly likely"?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/dannykopping/DebuggingChecklist/pull/1/files#r778871
You mean 'include passenger'?
This should say QUnit :)
could you fix the translation too ? It should be ``Ê``, not ``Ë``
Please remove all the auto-reformat changes... There are lots of these.
while you are at it, I would recommend disabling these auto stuff while saving.
Instead of commenting these out, just remove them ... you can always revisit the example to add them back in if needed later.
Delete this upgrade-related files.
It looks like the handler for REQUEST_LAST_CALL_FAIL_CAUSE reads two uint32s: `num` and `failCause`. Here you are only supplying a fail cause, but no `num`..

The logic in ril_worker.js is actually a bit confusing, the `num` field looks to be optional, but when it is not supplied, the fail cause isn't specified.. is this a bug in ril_worker.js?
https://mxr.mozilla.org/mozilla-central/source/dom/system/gonk/ril_worker.js#2929
Ok. I'll fix them.
Yes. You want [Array.shift] as documented on https://developer.mozilla.org/en/JavaScript/Guide/Predefined_Core_Objects#Array_Object
this should be ``html+jinja`` as you have some Twig syntax in it
we have some funky grammar here..  probably need to replace "The LingSync" with "LingSync"
this is where you say you get an error?
See above about '"control" in the process' - the idea isn't to see how many cool libraries they know or in how few lines of JS they can accomplish the same thing. It's about ingenuity, as well as controlled measurement of skills. It's hard to decide between 3 different candidates, if each uses drastically different tools to do the job.

By limiting the tools to most common denominators among CMG developers, the number of people who can help gauge applicants goes up, as does the relevancy of their opinions on a given applicant.
solved, thanks a lot
Ok, so we can flip to urllib2 when we make sure to add a CLI option to specify a proxy. This can happen in a follow-up issue. For now it's more important to react correctly on 404 responses.
> Would it make sense to open a new ticket to test that case of 2,000 patients and, er, however many procedures you'd expect for 2,000 patients, with an eye towards reducing that runtime?

No I don't think that is necessary. This change _should_ give enough of a performance boost. At the end of the day this is just an admin report and doesn't need to be super responsive, but it also shouldn't take 4+ seconds to run :flushed:
ok, i tried to set it to date, but i got a problem with pattern option now
@dylin Can we setup this profile so it doesn't execute on the SCM machine? They don't have a device attached to the machine so it would never be able to deploy.... There is a "scm" property in this file that we can probably make use of.
I will work on this after I complete the acal work.

On Mon, May 14, 2012 at 4:20 PM, Kyle Campos <
reply@reply.github.com
> wrote:

> > @@ -2611,7 +2611,43 @@ def dependency(coursename,  opts={})
> >        #NOTE: The following regexp returns the Program. NOTE: This
> regular expression will only work for the undergraduate Biological Sciences
> Program that has a Degree Type of Bachelor Science.
> >        # Due to the fact of duplicate programs of the same name;
> e.g.,Biological Sciences with a Degree Type of Bachelor of Science and
> Biological Sciences with a Degree Type of
> >        # Doctor of Philosophy exist, a regular expression would have to
> be created to handle the programs that share the same Program Title; i.e.,
> Biological Sciences
> > -      :item_name_var_regexp =>
> '\([^\"]+\)\"\,\"[^\"]+\"\,\"[^\"]+\"\,\"' + coursename
> > +      :item_name_var_regexp =>
> '([^\"]+)\"\,\"[^\"]+\"\,\"[^\"]+\"\,\"[^\"]+\"\,\"[^\"]+\"\,\"[^\"]+\"\,\"'
> + coursename,
> > +      :componentIds_dyn_var => 'componentIds_dyn_var',
> > +      :componentIds_var_regexp =>
> '\"lu.resultColumn.luOptionalDependencyRequirementComponentIds\"\,\"([^\"]+)',
> > +      :bsci360_dyn_var => 'bsci360_dyn_var',
>
> Why are you using references to specific courses in the variable name?
> This looks like there should be an array of hashes used here, we shouldn't
> be referring course names in the variable name itself.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/kcampos/Open-Performance-Automation-Framework/pull/24/files#r820225
>