labofsahil/github-event-dataset-2012 huggingface dataset:
No, it doesn't ignore it.

```
ruby -e'STDOUT.close; puts "hello"'
-e:1:in `write': closed stream (IOError)
	from -e:1:in `puts'
	from -e:1:in `puts'
	from -e:1:in `<main>'
```

Do we need to close the IO?
should find this issue PROJECT releases
I can't tell if this is a github interface weird thing, but there's a `+` at the end of this url. Does that seem right?
You can use `Assert.true` here.
This should be named something else. we already have a cls.mozillian
After thinking about it, I prefer the verboseness of setting the status here. Less magic, easy to understand.
These tests could have some simple docstrings.
I think the idea was that `.destroy()` should be responsible for making sure those are all removed.
Only bit of advice here would be that in Ruby, it's pretty standard for methods that return true or false to end in a question mark, like: def hide_hit_counts? That just makes it a little clearer to someone who's looking at the code that we're checking a value rather than performing an action.
one more thing -- after a quick look at the authentication concern you wrote above, i wonder:  can't you just manually pull out the specific keys needed for authentication and push those into the ByteArray first?  then, remove them from the object you're parsing into the ByteArray, and serialize the remainder of the document object in a non-determinate way (e.g. for-in).
:bool ?
I actually had a strange interaction with the former: `'wildignore'` is applied to this backtick-expansion, so it can have unwanted side effects ("E480: No match: \`=file\`"), so prefer `fnameescape()` in plugins, and just use backtick-expansion interactively in Vim.

In my case, I disabled a local filetype-specific configuration by appending `.bak`, but that one still matched my `g:localrc_filetype`, then caused the E480 error, because I have `*.bak` in `'wildignore'`.
Definitely not document - it's not mapped to CSL as an item type so it's generally a bad choice for people to store data in.
I would actually fall back to articleJournal, just because it has the most fields and is common. The alternative would be report or manuscript, which are good for "weird" types of documents.
yes that's the idea. Actually looking at this now, it doesn't seem right that this relies on the author coming first. I think it does do that in MARC, but we shouldn't rely on it - I actually thought I had re-written that once so that it didn't anymore. Let me check if I have code for that somewhere.
Maybe you mean `=` instead of `==`?
Hmm.. It looks like it only returns something else than GUI resolution after we have managed to go fullscreen. So maybe check it in CXBMCRenderManager::Update
I am sorry for the comment. I don't like being snarky, even if it is Friday
and I am under a time press.

I don't think that getEntity could cause any troubles since it is only
called when the expectation fails so 1) the teast doesn't continue for an
AssertError is thrown 2) it wouldn't anyway contain what the test expects.

I agree that having an explicit, readable assert is nicer. I just felt the
need of helping to troubleshoot potential failures more important than
beauty.


2012/11/2 Vidar Larsen <notifications@github.com>

> In
> restapp-bootstrap/src/test/java/org/cloudname/example/restapp/rs/AbstractResourceTester.java:
>
> > +    public final void setUp() throws Exception {
> > +        super.setUp();
> > +    }
> > +
> > +    /**
> > +     * Verify that the response's status is as expected, provide useful error message if not,
> > +     * including the response body (that might carry some details of the failure).
> > +     *
> > +     * @param expectedStatus
> > +     *            (required) What status you expected; example: {@link Status#OK}
> > +     *            (Use JAX-RS' Response.Status, not the Jersey-specific ClientResponse.Status.)
> > +     * @param actualResponse
> > +     *            (required) The response, from st. like
> > +     *            <code>{@code resource().path("exampleResource").get(ClientResponse.class);}</code>
> > +     */
> > +    public static void assertHttpStatus(final Status expectedStatus, final ClientResponse actualResponse) {
>
> I was merely pointing out a few issues with this kind of convenience.
> Please save your snarky comments for your blog, even if it *is* Friday.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Cloudname/cloudname/pull/80/files#r2017140>.
>
>



-- 
*Propaganda in signatures sucks but Clojure is really cool. (And
productive.) :-)
(Vær så snill og hjelp meg med å korrigere min norsk, skriftlig og muntlig.
Takk!)*

Jakub Holy
Solutions Engineer | +47 966 23 666
Iterate AS | www.iterate.no
The Lean Software Development Consultancy
- http://theholyjava.wordpress.com/ -
Is the return for this method used anywhere. If the latch already exists (e.g., if `putIAbsent` doesn't put the `countDownLatch`), then this method will not return the existing latch.
Crap, I made angels cry again.
you can use <context-param> element to share configuration between multiple filters or servlets.
please do not ask the transport, this will re-read the namespaces for each request.

the best would be to call Session::getNamespaceURI($prefix) . it will throw the correct exception for you.
I just wanted to throw in that page_by_destination has been helpful to me in a few of my extensions, I can tell this pull is still in flux but wondering if your planing to keep this (or some equivalent) public?
The text is changed temporarily because Browser ID is disabled. We should @xfail this test until BrowserID is back.
some stupid message
this is the same line as the commented one, is this normal ?
You're already checking for this with the assert on line 69
Right, hard-coded values. The current approach is fine then.
Just verifying: This will return a flat list of comments and the UI will have to construct the tree manually? (Although that'll probably be fairly easy as they are sorted in depth)
`tmp` doesn't seem to be anything
You should make this variable as `static`.
The indentation looks odd. Are you using spaces instead of tabs?
Could very well be:

    return ($this->wordwrap) ? $this->word_wrap($this->alt_message) : $this->alt_messasge;

... and the same in `_get_alt_message()`.
Are tables being escaped here?
Needs to be in parentheses. Furthermore, the whole rule can be written like this:

    return ! (isset($this->_field_data[$field]) && $this->_field_data[$field]['postdata'] === $str);
@slebedev Улыбнуло. А еще можно делать так:  Cell *cell1 = [[Cell init] alloc]; - тогда будет вообще громадный Failed. 
Вы предлагаете все пихать в один метод и тем самым делать методы не читабельными ?
Remove received packets from `writeBuffer`
When you submit a PR, any commits pushed to the branch that the PR was submitted from will be added to the PR
This one was my fault but could you add a new line here as well
I needed to add the `key=unicode.lower` when switching from sqllite testing to mysql.
Go ahead and leave the empty line then.  Thanks.
But making the signals sequential might not be desirable. Anyways, we have bigger fish to fry. I'll remove it.
Don't use Ruby 1.9-style hashes here. We want to preserve compatibility with Ruby 1.8. It's best to run the tests with both Rubies before pushing.
I'm somewhat reluctant to suclassing, let's weigh costs and benefits. What
about simply improving the epochs.copy method in a way that it doesn't copy
unnecessary stuff?


On Thu, Dec 6, 2012 at 3:57 PM, Christian Brodbeck <notifications@github.com
> wrote:

> In mne/epochs.py:
>
> > @@ -463,19 +493,30 @@ def __repr__(self):
> >      def __getitem__(self, key):
> >          """Return an Epochs object with a subset of epochs
> >          """
> > -        if not self._bad_dropped:
> > -            warnings.warn("Bad epochs have not been dropped, indexing will be "
> > -                          "inaccurate. Use drop_bad_epochs() or preload=True")
> > -
> > -        epochs = cp.copy(self)  # XXX : should use deepcopy but breaks ...
> > -        epochs.events = np.atleast_2d(self.events[key])
> > +        epochs = self.copy()
>
> Yes, hence "eventually" :) If you have a NewEpochs class that initializes
> with data, then the current Epochs could be transformed into a function
> that create such objects to keep the api backwards compatible (or subclass
> to make type-checking work as well).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/229/files#r2335632>.
>
>
change if statement to ```if(!filter || wordFiltering(story, filter))``` in this way any value of filter that would evaluate to false would be considered.
if (containsAddress) break;
If the other way is old and busted please delete it from the guide.
``getManager()`` is wrong for the 2.0 branch. It has to be ``getEntityManager()``
use fn arg requestUrl instead
The indentation is a bit wrong here.
Ah, ya entendí a qué te referís. Es verdad, la única razón que lo necesitaba acá es para poder bindearlo en el UI. Debería cambiar eso por recalcular en JS. En este punto era la solución fácil.
Good point, fixed: 5d857fc
Safety, and following the general pattern of getters() in the class.
objects are instances of a class. I'm not sure what you mean by subgroups.
let the comments breathe! space above, please.
Could you break this up into a few lines?
We probably have the CPU cycles to spare though. :p
%r is overkill since len is only ever going to give you an int or a TypeError. Try %d!
It's originally a Java convention but it seems like it pops up from time to time in Python land. I agree with @ed- that something called "check" I expect to raise an exception rather than return a valid.
Since segments is the reunion of finishedSegments and unfinishedSegments, maybe you should skip the 'segments' field.
Yeah, that's pretty much what I was getting at in the outside comment. I can get behind such a guideline.
That's possible -- I was just trying to be backward compatible here with third-party backends that don't currently implement this value.
if we use the locator from BasePage why don't we also use the method from that class
In the pull request I stated that this uses a video that is made by another test.

In the future we will use a video file from the sdcard directory in the repository. I noted this in the pull request summary too.

So I'm going to suggest option 3: Leave it as it is, it's fine.
You should be able to use a locator that returns multiple elements, and then select the element you want to work on by using its index. For example:

    self._pick_wallpapers_locator = ('css selector', '#wallpapers img')
    pick_wallpapers = self.marionette.find_elements(*self._pick_wallpapers_locator)
    pick_wallpaper[4].click()
J'ai remplacé le event.getLine(0).equalsIgnoreCase("easyg") par ca, car le equalsIgnoreCase n'empechait pas de placer un panneau avec écrit [EasyG] dessus et donc n'importe quel joueur de changer son game mode.
I would use resp.success() here instead
yes, it is a boolean type
Is it a gap or a tab character?
This line is not needed as _inOverworld is already set below.
Thanks for adding context to the translation strings, very considerate :)
there's no indentation here ;-)
Same deal with the `:::php` here, gives a nicer formatted code block in the formatted md file.
we should probably make that "1" explicit. I think I used that because of the border being 1px, but do we want to make that some kind of "bv = t.css("border-width")"?
Can we have a better named top level tag? ie. KML starts with <kml>
RSS starts with <rss>
Maybe <UshahidiXml> ??
I'll take a look at that change, I was just looking locally at the cases in the switch.
We're never freeing this allocated object on error.
I was wondering the same thing.  Can probably be changed.  Worth investigating...
move it to Connection
Porque agora adicionei uma resposta 404 no caso de não existir qualquer resposta. Nos testes estava à espera de sucess.
That's a good idea then.
Just to confirm, signing in this example was removed b/c it's actually not needed for this endpoint?
Ok, no problem !
nitpicking: `and not @line` is already covered
yeah, good call.  Sorry, this is my fault after spreading the dev over such a large window
Definitely SDK-internal. Package access please.
Well it isn't exactly the same. And it could be just the other way around, I would've change it and you would groan why did I do something you didn't want(it is not that I didn't notice it).
And `button.auth_button` matches only the logout button?
But here is OK if it blows. We want this data in there. So use `foursquare_data['response']['user']`.
Why defining this function if you are calling it only once?
I really don't understand why are you resizing titles? I thought that we said that we will use class for form fields?
Please make var_names a required argument.
Put a space between the method call and the block to make it more apparent that they're two separate things. Als o in the two cases below.
Ah! Excellent catch! This is why code reviews are needed!
ich den könnte man auch weglassen.
However this is true. I'll change that :).
Should receive the options object...
Can you add a comment for this workaround? Thanks :-)
> Clustering: deploy multiple Infinispan nodes

In this chapter I find it confusing that in the mist of the discussion of Infinispan configuration options (set in the infinispan xml) we talk about jvm startup paramters for IP version and network devices. I think it needs to be made clearer that the latter are parameter which need to be passed to the JVM at sartup
No worries, we never get it perfect either when reviewing each other's
code, but you got it close to perfection this time. Very appreciated!

On 28 June 2012 09:37, Seiya Kawashima
<reply@reply.github.com>
wrote:
>>       }
>> +
>> +    public EntityKey(String tableName, Serializable id, String entityName, String[] columnNames, Object[] values) {
>> +     this.table = tableName;
>
> I apologize on this issue. I knew how important following the code formatting was from another pull request, but the issue occurred again. It won't happen again. I formatted the part and committed as in the commit, [Format white spaces in EntityKey class ](https://github.com/seiyak/hibernate-ogm/commit/bb9876ba4dc3191a1cf15b5b04cb570f3b166c30).
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hibernate/hibernate-ogm/pull/98/files#r1066388
Double space!!1
Would you consider using the elaborate Bootstrap.php even after you expressed your dislike against Bootstrap.php::getServiceManager()?

I ask this because to test the view helpers (but probably others) it is necessary to have a multiple parts of a mvc context configured.

A service manager (one) that contains; a router and routematch (for url viewhelper), viewhelpermanager (which is a servicemanager itself). view helpers need to be injected with a view to be able to call other view helpers.

I know this can all be manually setup but this is quite tedious and error prone. This need to be repeated between the various tests (although that could be abstracted to a super class), and as this module (probably) will only be run in a fully configured mvc context I would prefer to avoid such a time consuming task. Let's a Bootstrap that does all this for us and be done with it..?
I get no warnings / errors from ant either:

[ivy:retrieve] :: retrieving :: com.twitter#elephant-bird [sync]
[ivy:retrieve] 	confs: [compile, test]
[ivy:retrieve] 	103 artifacts copied, 0 already retrieved (44212kB/266ms)

generate-protobuf:
     [exec] Result: 1
    [apply] Applied protoc to 4 files and 0 directories.

compile:
    [javac] /Users/hazen/workspace/elephant-bird/build.xml:85: warning: 'includeantruntime' was not set, defaulting to build.sysclasspath=last; set to false for repeatable builds
    [javac] Compiling 4 source files to /Users/hazen/workspace/elephant-bird/build/classes
    [javac] /Users/hazen/workspace/elephant-bird/build.xml:85: warning: 'includeantruntime' was not set, defaulting to build.sysclasspath=last; set to false for repeatable builds
    [javac] Compiling 165 source files to /Users/hazen/workspace/elephant-bird/build/classes
    [javac] Note: Some input files use or override a deprecated API.
    [javac] Note: Recompile with -Xlint:deprecation for details.
    [javac] Note: Some input files use unchecked or unsafe operations.
    [javac] Note: Recompile with -Xlint:unchecked for details.

BUILD SUCCESSFUL
Total time: 13 seconds
This method name and its reader should be more descriptive (i.e. writeObjectToConfAsBase64) to allow for other methods with different serialization schemes (i.e. writeObjectToConfAsJson).
Take this out. Instead, use Data.Conduit.List.map in your own code.
This is awesome, thank you so much. @madmatah had provided an identical patch earlier, so I landed his PR. Thanks again.
I tried that, but this this feels like the more logical order for the arguments IMO. I don't want to reorder the arguments based on how I call the function.
I don't think so. It's a method, but it's more or less the same as doing `foo.bar`, it has almost the same performance.
If you float a widget the floating flag only gets on that widget, not its children. So if this widget is floating, then its container is `FloatContainer`, which is a magic internal thing that we don't want to send events to. So we stop propagating events.

Another way of writing it would be `if (GetContainer() == GetContext()->m_float.Get())`, except `m_float` is private.
I feel this should just use an if else for style points.
Should we remove the XARGSARG here? also from the comments to of the code, they are unused now
So, what /is/ the ideal way of doing this.
There is os.rename, shutil.move, etc.
To explain a bit more (for my own edification), what it looks like you have now is a sum of each leave one out density estimate (not-normalized by 1/(n-1)) evaluated at each left out point. This doesn't seem right to me.
https://github.com/swiftmailer/swiftmailer/pull/233  thanks
We decided to eventually use symbol keys in all our yaml resources.  I commented on the use of symbol keys here because none of our other resources use them yet, so it seemed inconsistent.
The type should be a{ss(ao)}
I think the does not belong here ;)
Similar to above, I don't think all this repetition is necessary.
I did some testing with ZendServer, it's packaging is kinda suboptimal (in terms of dependencies), but in almost all scenearios the /usr/local/zend/bin directory exists, even after removal of Zend Server. Therefore, I would propose to check for /usr/local/zend/bin/php instead.
No, it's not a big deal and you're probably right, so we should leave it.
I think that's largely an improvement, but I still think of Postage stamps as being separate entities so in future I would vote for

PStampImage -> SingleImage
OverlappingImage -> OverlappingSingleImage (or just consider these two as SingleImage)
SeparateFiles -> SeparatePStampFiles 

...admittedly at the cost of longer names.  This is all just a matter of subjective opinion/prior experience of course!
We use beta for the parameter of the Moffat profile and for position angles.  Need to specify more explicitly which this is?  Or is the fact that there's (e1, e2) enough to indicate that it's the first kind of beta?
Setting __class__ seems to work at least.  But I accept the criticism that it may be bad form.  (I don't claim to have a handle on good python style yet.)  When I wrote the `__iadd__` function, I didn't realize that it wasn't required -- that python would sort of do it automatically using the `__add__` function.  So I agree that it would be better to just remove the `__iadd__` function and let python implement the automatic version.  

The other places we do this are:
1. when we apply a transform to the SBProfile, which turns the SBProfile object into a SBTransform.  But python doesn't know this, so we set the `__class__` to be `SBTransform` so errors will produce a more sensible exception at the python layer, rather than a crash in the c++ layer.
2. when we implement GSObject `copy` method, where we set the `__class__` of the returned object to be the same as the original.

Are either of these poor form?  They seem less of a problem to me than the iadd usage.
On May 21, 2012, at 5:42 PM, Rachel Mandelbaum wrote:

> Could we imagine a circumstance where the requirement that the ```PhotonArray```s have the same size would be problematic?  An example I could think of would be if we have a complicated galaxy requiring tons of photons to represent properly, and a fairly simple PSF that is easily represented with a factor of 10 fewer.  Do you think this is unlikely to happen much in practice?

Yes, unlikely: the `PhotonArray::convolve()` method should not be used directly by user code in practice, it's used within `SBConvolve` and within `SBInterpolatedImage` (in the latter case to convolve the pixellated image with the interpolation kernel).  It doesn't make sense to call it with different length arrays, so if a user does this, they've made an error and it's appropriate to throw the exception.
Maybe should add that Lanczos is the usual compromise.  It is a Sinc interpolant multiplied by an envelope that cuts off the high-x values relatively smoothly.  [Wikipedia](http://en.wikipedia.org/wiki/Lanczos_resampling) claims that:
> The Lanczos filter has been compared with other filters, particularly other windowing of the sinc filter. Some have found the Lanczos (with a = 2) the "best compromise in terms of reduction of aliasing, sharpness, and minimal ringing", compared with truncated sinc and the Barlett, cosine, and Hann windowed sinc, for decimation and interpolation of 2-dimensional image data.[2]

This might be worth putting into our documentation either here or for the Lanczos class.
Those 'STOP's could be put in the task_queue before starting the processes, but it doesn't actually matter.  The way to think about this is that Process command starts up a parallel process that keep checking the queue for a new task.  If there is one there, it grabs it and does it.  If not, it waits until there is one to grab.  When it finds a 'STOP', it shuts down.  So you could even put the 'STOP's on the task list after the next loop, or even at the end of the script.  You do want to make sure to put them somewhere though, since the program won't stop as long as there are running processes.

In the meanwhile, the main process keeps going.  So in fact the logger stuff below is happening while the other processes are still working on other tasks.  In fact, since the below loop pulls off all the results from the done_queue, it serves to sync up the processes.  So you could wait on the STOP's and do the two write commands in parallel too by adding them to the task_queue _after_ the following loop.  (Although you'd have to change how the worker function is written slightly to allow either kind of task on the queue.)
It is good practice to declare global variables at the top (module level) of a script.
bad bad.
not a big deal, but because you're already passing `exam`, even in the template you can write
`exam.xml_metadata` where you now write `question_times`.  I.e. you can do instance field accesses in the template language.
decided to not do any endtime-based restrictions on the collection endpoint.  let me know if you want me to change this behavior
Would it kill you to add space in the assignment statements? ;)
Shame on me, we mentioned this during the Hackergarten!
Why not `assertEquals(result, ImmutableList.of("hello \"1\"", "hello \"2\"", "hello \"3\"");`, rather than just size and first element?
System.err does automatic flush. That's needed for System.out
Não existe mais model.Publisher

Os testes já foram removidos em commit anterior, realizar merge de código.
Would you be able to use a function decorator for measuring the timing instead of a with-clause?

That would avoid the 'logging.debug + with' boilerplate that's everywhere and still give you the same functionality.
а. ок
It will select all of them by default.
Does ``select_related`` have any effect without specifying the related models to load?
This looks like it's Guzzle 3.0.  If so, we need to correct composer.json.  Also, we should probably take my PR first since it's still based on 2.8 (though I realize that's a bummer).  Otherwise, that PR would just need to die.
I don't think that's necessary here, but I would be fine with it either way.
Revert these examine changes.
Counselor is a subtype of Chaplain, and should go under "1" as well :|
The referenced approach look good to me - are the  remaining issues something you can get past (either with or without assistance)?  If they look insurmountable, then we can just leave what you have for now.
<span><%= nth %> 回目</span>より<span><%= "#{nth} 回目" %></span>の方がすっきりするかな、と思いました。
Это лишнее. Надо просто убрать catch блок.
У тебя в тесте вообще Generics не указываются.
yyyyyyyaaaa
you should have an at-param here describing what a "XUL tab" is.
Generally exceptions are expensive operations and as the name would suggest, exceptional conditions. So using them in the context of controlling the flow of your application is indeed considered bad practice.
Would have been much better if you first queried if given ID exists rather then waiting if first operation fails so you can try the other one.
We still might want to ensure it's an actual project wall post and throw an error if it's not. This would make a tighter API because allowing clients to update and retrieve any wallpost through the project wallpost api is exposing an implementation detail to the client. But I guess I'm just being pedantic. This is good as is.
as far as I understand, this doesn't invoke the garbage collection, right?
So is this really necessary?
Reading one more time the Ping Li, and et al paper, his recommendation is 1 / sqrt(n_features) and not min(1 / 3.0, 1 / sqrt(n_features)).

If the user wants to use 1 / 3 for n_features < 9, he still has the possibility. So I would prefer to set the default to 1 / sqrt(n_features).
See  f74c502
Hum ok namespace class issue...
Also, we cannot modify the parameters in an __init__, elsewhere it will break clone (by design, as it is against the API of the scikit). We have to do it in the fit. I am on it.
phantom whitespace (also see L130)
Please change your IDE setup so it doesn't add this to jbossas classes.
Probably a mistake during merge/conflict resolve. I'll ask vladimir (who have written test cases) to have a look
Shouldn't this stay as 'puppet' ?
Also add the following line please:

`The activity indicator will choose the largest of its three graphics sizes that fits inside the avaliable space.`
Also here, use methods `isSingleValuedAssociation` and `isCollectionValuedAssociation` from the interface
I like the name of this.  Better than isStringOrUnicode()
I've been adding short forms on a case by case basis, with more common parameters more likely to have a short form, but can definitely add support for -p.
Looks to me, that assert can never be hit..
t @name by definition == 'superfile', otherwise wouldn't have returned from getNamedPropTree, I think.

Probably should be an error here, if t && link, implying it's already part of the super...
but I think that will be have already been caught by addSubFile
why add queryDfsXmlBranchName(DXB_Internal)) to 'sb' and not direct to 'out' ?
Doubt it would make any difference, and made the code a lot simpler.
Yes, good point. I think that the first solution is cleaner.
[`dict.pop`](http://docs.python.org/2/library/stdtypes.html#dict.pop) allows you to specify a default value that will be used if the key isn't in the dictionary:

    bind = options.pop('bind', db.engine)
We have three choices here:
- 1/ use the same way to pass arguments as in many redis-py methods: *args and **kwargs, and so have consistency
    ```python
    foo, bar = my_obj.hmget('foo', 'bar')
    ```
- 2/ use a way used only for hmget and hmset in redis-py: list and dict
    ```python
    foo, bar = my_obj.hmget(['foo', 'bar'])

You can tell me which one sounds better to you

And the third way: 
- 3/ accept the both syntaxes, to let people easily come from a non-limpyd source code to a limpyd one, byt to let others don't be mad with inconsistency

So tell me your choice, i'll accept it. If it's not 1 or 3, i will override it in my own source code
it has to be /[a-z]{2}(_[a-z]{2}){0,1}/ (i.e. with _ not -) so it should match  en, de, en_us, ch_fr, etc
Shouldn't we do the same than menuitems-jplib and give a link to a precise tarball revision, instead of project home on github?
It feels safier and easier for people not used to git/github.
callback.apply(callback, arguments)
This is another place with the 1.9 hash syntax.
Wouldn't it be better to use [`django.test.TestCase`](https://docs.djangoproject.com/en/1.4/topics/testing/#django.test.TestCase) instead of `unittest.TestCase`? This would avoid having to write this setUp tearDown, by using [test fixtures](https://docs.djangoproject.com/en/1.4/topics/testing/#topics-testing-fixtures)
Voiko olla totta, ettei tätä funktiota oltu vielä tehty o_O
IIRC, currently we have 2 shortcuts here: if the tile accept no content type then this method should return **[]** (an empty list) ; if the tile supports all content types the this method returns **None**; I think that's the way it has been done in list tile and probably we should find a better way to do it.

@flecox @frapell @Quimera comments on this?
Inception comes to life, we're testing this file itself
Which part? I'm assuming the footer. I wanted to remove the other pieces but not sure what the UX Designers had intended to do with the widget so I was going to change the display then
I'm using `.value`  to handle the case where `self` has a nil value (in the case of an empty list).
@brandonprry Just reviewing HACKING: 

9. Never, ever use $global variables. This applies to modules, mixins,and libraries. If you need a "global" within a specific class, you canuse @@class_variables, but most modules should use @instance variablesto store information between methods.
Shouldn't you be using the url parameter?  Not hardcoding it.
Version is not required anymore for modules
FWIW, the [paper referenced](http://web.mit.edu/persci/people/adelson/pub_pdfs/pyramid83.pdf) in the docstring also uses expand and reduce.
Really? I see that pretty often in our pre-built apps - it's certainly there in the existing Salesforce app
I see.  Yes, actually that one second timeout was tripping me up for a while before I figured out what was happening.  Thanks for the clarification here.

I also changed the wording in a few places to read more like that mnemonic.
This should probably be as part of the validate_resize_instance
No inertia on tilting?
This will ignore an outline width of zero.  a || b strikes again!
缺少参数解释……
Sorry, I did not notice this is excludes section. Please ignore my previous comment.
I don't know how nit picky I should be since this is already old and published, but here is my comment...
So, either the comma should be a period and you are missing periods in the last three bullet points or remove the end punctuation, which seems to be precedent.
Not strictly true. IOS allows for customization on the placement of paging control. See `pagingControlOnTop`, `pagingControlAlpha` and `overlayEnabled` on how to customize the paging control on IOS
I think we can actually remove the iPhone 3.2 restriction here, because we don't support iOS earlier than 4.0... I think. Need to clarify whether SDK version 4.0 (our minimum) means we can only run on iOS 4.0 or newer...

Also, I think the modal or full-screen window limitation may be specific to iOS. Would like someone from iOS to weigh in on this one to make sure the description is correct.
I think we need a more explicit error here, because it;s quite common to configure several servers. Sync itself points to several memcached server no ? or has it moved to a single local membase server ?
that makes sense. The second form should still work in both cases though, right? We're requiring an absolute path based on the location of this file...
why doesn't this use `vms['blimpy']` like the rest of the hypervisor sections?
@hunner Sorry about that. I amended the commit with a more descriptive message.
I won't comment on this. Wait for Itay. Can't remember how JavaScript iterators are supposed to work and I don't have brain space for it now.
the link points to your repo.
And yes, this line was called, it was the whole reasong for the jackrabbit lock-up ;) See https://issues.apache.org/jira/browse/JCR-3244 for a reproducable phpcr script
Never called, get rid of it.
`STATUS_FAILURE`.
But in the addInsert, addInserts and so on I will stay with object, since otherwise the user would have to check and cast the models on his side.
I'd be interested why you change this, what was the problem with using an `ofColor`?
It thought I got that one! Done. https://github.com/silverstripe/silverstripe-cms/pull/169
`el.each`
This should be `from .models import ...` to be explicit about relative vs top-level.
Tags shouldn't be required, I don't think.
For the attribute, I will use result-type (to be consistent with the marshalling transformer)
Hmm ;)
I did missed it. But i guess I still like the no "go to" approach. Funny when Mark was looking at it he suggested boolean as well.
And yes I replied in the previous that I did fix that.
You don't need to call self here --- if you just say `stand`, it'll call the local method on the current object
What if the trigger is qrcode?

Higher level, why not just use handle remainder?
@christianv Perhaps a little OCD, but can you add a space between the hyphen and DESCRIPTION_OF_WORK ?
See comments for ConstructorDescriptor
this file should also be inside the ``en`` folder
I actually wondered that myself.
The method would be named `areParametersConstrained()` and I did feel like it was less readable than `isConstrainedOnParameters()`. Also, I tend to find methods starting with the same prefix for the same feature set easier to use upon auto completion and doc scanning generally. 
But I don't have any strong holds. Do you think `areParametersConstrained()` would be better?
is this just indentation chnage?
Fixed & PR sent.
in the multiclass case, if the labels are not uniformly balanced in terms of size, will this do sampling with replacement? perhaps one could raise an exception if someone is asking for a test size that cannot be met?
It would be nice if we didn't have to do this to run the factory, but I don't have any immediate ideas.
I was hoping with the move to ActiveRecord queries we could avoid doing this summary / count code in Ruby. That might mean this generalized method goes away, but for procedures we could do something like:

```ruby
scoped_procedures = date_time_where(PatientProcedure, 'patient_procedures')
procedure_count = scoped_procedures.count
procedure_value = scoped_procedures.select("SUM(procedures.cost) AS total_cost")
  .joins(:procedure).first.total_cost.to_f
```

The more I see / use that date_time_where the more I hate it :poop: That's my fault
That's no problem. I'll update code later. Thank you for all your reviewing.
listFiles function would pickup those files. It has a recursive read directory call to wrench.

----- Original Message -----
From: Nukul Bhasin [mailto:reply@reply.github.com]
Sent: Tuesday, May 15, 2012 06:02 AM
To: James Keshavarzi
Subject: Re: [BB10-WebWorks-Framework] Pull req - Fixed issue #95 [Left over build files cause Path errors in Windows XP] (#34)

> +    if (path.existsSync(extPath)) {
> +        //Iterate over extensions directory
> +        fs.readdirSync(extPath).forEach(function (extension) {
> +            var apiDir = path.normalize(path.resolve(extPath, extension)),
> +                apiDest = path.join(extDest, extension),
> +                soPath, soDest,
> +                apiNativeDir,
> +                jsFiles, soFiles;
> +            
> +            //find all .js files
> +            jsFiles = utils.listFiles(apiDir, function (file) {
> +                return path.extname(file) === ".js";
> +            });
> +            
> +            //Copy each .js file to its extensions folder
> +            jsFiles.forEach(function (jsFile) {

what if there was a folder inside the extension that had more JavaScript files in it?

---
Reply to this email directly or view it on GitHub:
https://github.com/blackberry-webworks/BB10-WebWorks-Framework/pull/34/files#r822215

---------------------------------------------------------------------
This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.
should you abstract this read method out?
Would it make more sense inside ppsUtils?
Don't believe there is. Let me see if i can write something up really quick.