labofsahil/github-event-dataset-2012 huggingface dataset:
Maybe I'm the only one but I'm a little concerned about the memory overhead for a simple coordinates class. Instead of caching the transformation is it feasible to check if the attribute b has been set yet, then if not then set b as the output of the external transformation function. Anytime someone tries to access b later, the attribute has been set. This would be a problem if you changed your .ra, .hms values etc because the .b attribute wouldn't necessarily update automagically. 

Thoughts? I just think 160mb of memory to store ONE transformation for 10^7 sources seems a little absurd. 


On 15/08/2012, at 10:09 AM, Wolfgang Kerzendorf <notifications@github.com> wrote:

> In api/coords.py:
> 
> > +    as separate values or arrays.  Inputs like "180.25, -23.5" are
> > +    not allowed, but this greatly simplifies the API.  (IMO)
> > +    """
> > +
> > +c = Coord(90, 270, 'deg', 'deg', 'equatorial')
> > +c.ra  # 90
> > +c.dec  # 270
> > +
> > +c = Coord([90, 180], [270, 271], 'deg', 'deg', 'equatorial' precision=0)
> > +c.ra  # [90, 180]
> > +c.dec  # [270, 271]
> > +
> > +c.galactic.b  # whatever that would be
> > +c.b  # Do an implicit transformation to the galactic scale.  Not
> > +     # sure if there are difficulties with this.
> > +
> @taldcroft: I think here's the deal: Let's not worry about memory limits - if you have 10^10 sources and use the coordinate class then things might break, but let's hope that's less than 1% of our user base ;-). But this got me to an important other point: These transformations should be accessible without using the coord objects. I'm not saying easily, but accessible. We can then put in the documentation saying that you can use the raw transformation and use it on your terabytes worth of data.
> 
> —
> Reply to this email directly or view it on GitHub.
>
These should all be relative imports such as `from . import models`
这个我完全同意，所以Request是个挺大的事情。
what about adding content type detection here based on the name?
This should still not be on client-neutral bitcoin.org
This change seem wrong for me, does it have a reason for doing that ?
Let's defer that conversation, then.
Fixed on b6ff247
"mainstain" Spelling?
@stof skip it for now :)
Could you move this to the top of the file, and use ``require_once``?
My thinking was more along the lines of this:

The pipe is a just a conduit to pass messages to the application in an easy way. The application will decide how to send those messages to a client. In that fashion the pipe interface isn't a *real* pipe (though I suppose it could be) and the application will choose how to serialize messages. 

The thing that worries me about this approach is that the number of methods we'll need to define on the pipe is potentially large. For now, its just a method to handle encoding binary data. But what about other kinds of data that needs to be serialized?
if not rowset:
        return {}
Shall we catch this error then rather than just crashing?
This file got here by accident, it's not supposed to be here. In fact, the whole `scripts/` directory under `static/` shouldn't be there at all.
I'll add to this discussion by saying that we have our own reimplementation of project_solution for many of the same reasons (efficiency and calling our own custom objects).
That doesn't work, because neither `LIBGIT2_VERSION` or `")"` are wide strings (said the compiler when I tried this). There is a macro `TEXT()` that's meant to do this for strings behind macros, but it only makes the text wide if you `#define UNICODE` at some magical point. We could probably do this in our MSVC compat header without breaking anything else but I used this so it would definitely work in the first iteration.
this is gone!
Move this regex to the top of the file
I'm not sure it matters in this case, but we do have a .inline-block(); LESS mixin that outputs the IE fallback hacks too.
ok, who are the product people?
This semi-colon isn't needed.
it's `timezone = 'UTC'`
From what I have tested it does not seem to be an issue as everything works as expected. If you feel there might be an issue down the line, I can always change it to report_products
@tbaum can we check that this doesn't break anything on heroku?
Might be better to use $this->useTag() instead
This is great but you got rid of my default state! Can you just leave it in there and I'll find a way to load up the right default list based on their country choice.
I would recommend a for loop here. Like this you can check the counter variable later.
IC already taken, must be backwards compatible.
How does it work if you have more than one streamed sub-requests? The stream would be already closed for the second one.
Can you provide link please? Don't tell me it's PSR :)
Classmaps are looked first by the autoloader so it is faster (not really relevant when the APC autoloader is enabled of course).
Besides, IMO such cases are really the reason for the existence of the classmap: you have a class not really PSR-0 compliant (I find the current way kind of a hack) so you simply map it to the file it belongs to...
Another thing (although it would seem very weird...), the current way "forbids" any vendor to use the `SessionHandlerInterface` namespace (which would be horrible, of course, but still...).

Of course both ways work (same thing for the locale component) but I find mine "cleaner"... Don't know what @Seldaek thinks on that matter...
The CacheItem represents a single item. So it seems weird to have a default ttl in it. An item has only its own ttl.
I mean not debug mode. Log error and continue execution.
I didn't realize there was an interface, thanks! Fixed now.
`unset()` accepts multi values.
It's private --- should this be renamed to matcher?
+1 `StringUtils`
This is sueless as it already has a default value set in the property declaration
``` php
<?php
"0" == false;
```

same issue in `getSchemeAndHttpHost`
Depends on the error.  If the problem is that the request didn't pass validation, you want to send a 400 bad request error I think.  Unfortunately play doesn't have a handy method for that, so you'd have to set the response header manually.
if it's used as pattern, venue-bar image could also be only 1px in width - it would save some space
if ie dige lazem nadare
va nemikhad az noskheye printesh estefade koni, noskheye addisho bezar
Now you can add an empty public method propertyChanged() and call it inside propertyChangedInternal() so that users can subclass this method and override this method to get the callback when property changed.
Vertically align elements: all words start at the same column (4 spaces after warping block indent), columns are align with the end of the longest word.
Same comment as for TiffInput: Is this still necessary?
typically, even if the result is None, I would return the result. I'm sure it makes absolutely no difference in this case.
If you noticed, you would have got a warning now that this test exists along the lines of "A figure was created and not destroyed in a previous test". All tests which make a mpl figure should close it (unless they are wrapped by an ImageTest).
ничем. показалось более explicit. С питоном знаком/играюсь пару дней всего, поэтому еще не знаю общепринятых идиом
I missed some stuff. It's hard to copy edit / format that much text :)
this might be the source of the current bug: datalist isnt loading after registering a new user. closing the window and then reopening it.
There is no need to both inject via @ArquilllianResource and then create a new on in this method.
Can I give the choice to the user to use relative path or full path ? 
The standalone script supports that :
```sh
./standalone.sh -Djboss.server.base.dir=../serverB
```
I could use the same method [1] to resolve the ```jboss.server.base.dir```.

[1] https://github.com/bstansberry/jboss-as/commit/c6902f388d2ab595229c93063f061ccec6120b89
`this.indeterminate = val === false;`
I noticed that this line was changed from _MCR to _ANVIL by seventh at https://github.com/mcedit/pymclevel/commit/b83b6d4ba7772444df229eb4dabd1b09667587f2 as part of https://github.com/mcedit/pymclevel/pull/105 but then seventh requested that it be reverted since it did not work.  What else has been changed to make it work?
I see. Nice trick.
Missing a check above this line. We are going to ignore modal prompt from non-window frames from now on.

so 

    if (evt.target.dataset.frameType !== 'window')
      return;
amm ok but I cannot assign to myself because i'm in another war :)
puppies!
What about moving this if upward to avoid showing the response form and if evt.data.sessionEnded exists hiding it again.
fixed in the other PR
-Sriram




On Dec 21, 2012, at 8:05 AM, Shih-Chiang Chien <notifications@github.com> wrote:

> In apps/system/locales/system.en-US.properties:
> 
> > @@ -239,3 +239,5 @@ newSimPinMsg=Create PIN (must contain 4 to 8 digits)
> >  confirmNewSimPinMsg=Confirm new PIN
> >  newPinErrorMsg=PINs don’t match.
> >  
> > +#captive wifi
> > +captive-wifi-available=The network {{neworkName}} was found. Join network?
> typo: should be networkName instead of neworkName.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
sorry for the silly question, but why do we have to skip the first callback?
nit:  | } else { |
If Settings doesn’t use these l10n entities any more, please remove them from locales/settings.en-US.properties.
Awesome.  Since /sbin comes first, and we can put whatever we want in there and it will override /system/bin.
Sure, I agree :) But as soon as I change the css include to 

<link rel="stylesheet" type="text/css" href="/shared/style/custom_dialog/custom_dialog.css">  

it breaks the installation process of Gaia and throws the following error:

run-js-command  webapp-zip
build/webapp-zip.js:212: NS_ERROR_FILE_UNRECOGNIZED_PATH: Component returned failure code: 0x80520001 (NS_ERROR_FILE_UNRECOGNIZED_PATH) [nsIFile.append]
make: *** [webapp-zip] Error 3

Is that possible that we would make a separate PR to move custom_dialog.css and related files to shared folder, because we would really need this feature now.
This tweak is needed cause I don't know there is also a localized event when the user switches languages, looks good.
This actually seems to pass with after_save too - what am I doing wrong?
I know it is the same, but i prefer `Time.now`
please use '...' (3 dots) or the HTML entity '&hellip;' in the placeholder - or no dot
Would you mind reversing this one too?
This exception seems pretty vague. Can't we give more information about what went wrong to help debug it?
Like for Cassandra and HBase, there's a lot of code copy-pasted here, which means you have to maintain three (so-far) versions of this code. Definitely extract the copied code to a superclass method.
instead of using ``&&`` which would run phpunit only when phpspec passes, I would use 2 scripts so that both runs (and the build is successful is both pass):

```yaml
script:
    - php bin/phpspec run
    - phpunit
```
``` coffee-script
domain = if m = link.match(/;show:(.+)$/) then m[1] else link.match(/(\w+)\.\w+\//)[1]
```
Comment that this test requires a working internet connection.
It looks seems to me that this block duplicates functionality we already cover in ``AnathemaReflections.getResourcesMatching(String namepattern)``.
By moving the item files somewhere below ``resources\data``, and handing in an instance of ``AnathemaReflections`` into the bootjobs, you could leverage them.
While this is no longer necessary for handling overwriting of "default" items, I'd keep this in here simply because it handles an edge case:  The user drops in somebody's `Equipment.yap` file in with their JSON items.  With this in place, it will preserve their JSON items if they're the same name as the ones in the yap file.  Without this, the yap contents would overwrite the JSON ones.
If "produces a notice" == "explodes" then yes, this will explode. :)
Sorry, my code normally isn't that unclean.
enable CC_TEXTURE_ATLAS_USE_VAO  !  

#if CC_TEXTURE_ATLAS_USE_VAO
    //
    // Using VBO and VAO
    //
    glBindVertexArray( m_uVAOname );
glBlendFunc(sfactor, dfactor) == glBlendFuncSeparate(sfactor, dfactor, sfactor, dfactor)?
do you really want to validate the schema on each run ?
So how would the version constraint look?
I'd emit the checkEvent rather than the check. The checkEvent has the event duration, the cause of the outage, ...
again, we'll have to fix this later
access to flow needs a flow lock.
esses unbinds.. não seria melhor deixar fora do if (e do else)?
`mix` is not used, no need to import.
but that's not in the docs :)
Please prefer standard `array.forEach` over non-standard `for each`
Move this to pick_email
@seanmonstar - I have added a comment[https://github.com/mozilla/browserid/pull/2455#issuecomment-8609628] to #2455 to address partials in client side templates. What'cha think?
@ozten - still using "proxyidp" here.
I find it helps to have a link to the Bugzilla new bug form with the important fields filled out already. For example, with SUMO, I use: https://bugzilla.mozilla.org/enter_bug.cgi?product=support.mozilla.org&rep_platform=all&op_sys=all
dammmmmmn! alright :)
@jeffweiss it shouldn't matter currently, but what if the implementation changes (again)? Then I'm not so sure. Given the grief pathnames with spaces can cause, I would much rather test that pathnames with spaces work, and be assured that pathnames without also work, then the other way around.
"This method" instead of "read"
This doesn't actually seem to work quite right, see:

```python
@interact([("n", ContinuousSlider())])
def f(**kwargs):
    print n
    print locals()
    print kwargs
```

as compared to this:

```python
@interact
def f(n = ContinuousSlider()):
    print n
    print locals()
```

Granted, it does the exact same thing on aleph right now, but it would be a nice thing to get working if we claim to support it.
No go in IE7—need an alternate approach.
Yes, right. 
But this is only place where is it used, so I think it actually looks more understandable than using a constant defined on the begining of the file.
Technically yes, but that is an API was only used when extending classes.  The new API is not only internally used but externally used, and is more generic.  Had we not done this, we'd have a ton of duplication.  Also, previously, we were using strings like 'select' and 'join' in places (like getRawState()).  Now you can use a clear and concise constant.

I think this is acceptable.
as far as I remember `md5()` now internaly uses `hash()` (wich is in PHP core), so I guess there is no difference
I just removed `function_exists()` check for hash
@blanchonvincent I've discussed this with @ezimuel today, and pointed out that it has become a common practice in ZF2 to mark static classes as abstract, but not to use the "Abstract" prefix; this prevents having dead code (you don't need to have an empty constructor), and the lack of prefix indicates it can be consumed directly without need of extension. It also leaves the door open for extension later (as moving from abstract to concrete is not a BC break).
Why so? I have tested it and it does not rise notice. Agree with the reset to the original value.
Agreed, that's been done.
`Fin'ora non abbiamo mai spiegato alla nostra applicazione che è stato aggiunto un nuovo modulo!`
Yep -- the instanceof check is the appropriate way to test for a more specific type.
New-line nitpick :)
This can return false positives. 'f**z' turns into 'fuck', even though that doesn't make sense (what happens to the z?). 
According to the original problem description, we only work about censoring things that are one letter, followed by all *s. 

Also, this messes up case. 'F***' is never de-censored.
Maybe change the ID to createMessage instead of "create message".  Haha, why is there a space in the first place?
You can get rid of all this, and just use the `call_user_func_array()` implementation.
What's the point of adding a getByName method?
page_name=tag.url)
This method needs a test.
well, the point is that you have to duplicate all listeners of the mapped superclass if you want to add an additional one currently (making it likely to break things if you don't look at the mapped superclass carefully when mapping your entity). A solution could be to add another parameter in the annotation controlling the inheritance. It should default to inherit them, so that if you set it explicitly to false, you are expected to understand that you need to check the parent.
Nevermind .. `nil.to_sym` will asplode.
Hmm, he made a point that it only does certain optimizations when "under pressure". And he slapped a while (true) in front of the function call, which makes sense. No need to prematurely optimize if it doesn't matter.
better yet, use the constants from the report class (and remove them in the class). these should be defined on only one location
Should be http://hpc.ugent.be (which, true, gets forwarded to the URL in this file, but still ...
The reason I put it in this order was because during the development 
phase, you often don't even start your server. You only build in order 
to clear up compile and build errors. It's not until you get to the 
testing phase that you actually deploy. But whatever you think is fine 
by me.

On 04/01/2012 06:16 AM, Pete Muir wrote:
>>
>> -            mvn jboss-as:undeploy
>> -   6. If you want to redeploy a quickstart that uses a database, you should undeploy it first. This ensures the database is cleaned up correctly before the redeploy.
>> +            mvn clean package jboss-as:deploy
>> +
> Probably I swap the order of this  - Build&  Deploy and then Build Only.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jbossas/quickstart/pull/167/files#r630609
Perfect. Except this should really be outside the with statement. No matter what happens with the file, lets make sure that we refresh the index.
Yep, it was actually pretty misleading because for the normal case, the cells *were* that width, but it was because the content was that width, *not* because of that line.
Capitalized "Server" to be consistent with other instances of the phrase.
There was a | separating the buttons, which looked rather odd because we don't use it anywhere else.
> border=やthはnonleyout tableという意味だったのですね・・・

それは妥当でない拡大解釈だと思うので、仕様を簡単に読んでみるといいと思います。
always required, if the system is managed
There's no need to check ```match.length``` here.
If ANGLE was 90, the expression  "graph.congruency.intersect" is not being evaluated, probably because slope tends to infinite in this case.
这个名字不好啊。。。itier的itier？ `itier.itier`
When I commented it just called `parent::inTransaction`
inline внутри класса не нужен
Looks like someone already added the necessary change, but forgot to update the fragmentAndSend call to use the updated message.
You're totally right...

http://git-scm.com/docs/git-pull

>Incorporates changes from a remote repository into the current branch. In its default mode, git pull is shorthand for git fetch followed by git merge FETCH_HEAD.

>More precisely, git pull runs git fetch with the given parameters and calls git merge to merge the retrieved branch heads into the current branch.
add another blank line
security issue fix http://blog.nodejs.org/2012/05/07/http-server-security-vulnerability-please-upgrade-to-0-6-17/
The underscore reference is unnecessary. Just reference the native `Array.prototype.forEach`.
Does this need checking if an admin is executing this?
In the content module, I've changed this to:

`{
        PRINCIPAL_TYPE_GROUP: 'g',
        roles: {
             MEMBER: 'member'
             etc.
`
Same as before (i.e., Why not keeping the access modifier?)
I'd indent the following lines two spaces, or line them up to the parentheses.
}
    else
    {

... and sorry about repeating myself, but documentation update and a changelog entry are really needed. Otherwise - thanks, it's a really nice addition to have!
забавно. А если NUCLEO_COUNT изменить, работать будет, но будет утечка, пускай и небольшая.
perc - процентный показатель. То, что в данном конкретном случае у класса Cell массив состоит из 100 элементов - счастливая случайность.
То есть, решений таких было много самых разных, и цикл с заполнением воспринялся как сам собой разумеющийся. Я например не перемешивал свой вспомогательный массив, а просто его заполнял, и удалял из него использованные элементы.
cell в данном случае это передаваемый этомум методу объект, а dna - это его переменная
если бы dna был объявлен как @property, тогда нужно было бы писать как "cell.dna", если просто переменная, то cell->dna
Надеюсь что я ошибаюсь, и тогда "я многое пойму", но:
В мною приведённом коде - да, всё как вы говорите, но я не пытался его сделать равновероятным.
Я только хотел поправить
  [DNA objectAtIndex:index]
на
  [DNA objectAtIndex: indexToMutate]
что, ясно было, есть просто опечатка.

Но заодно я захотел поменять "% 3" на "% 4" - и это и была "спешка вывода". Недопоняв я заподозрил опечатку и в этом.

Теперь, поправьте если я ошибаюсь - условные вероятности выбора новых элементов для вами приведённого кода:

P(X:Y) - вероятно выбора нуклеотида X при условии что заменяемый нуклеотид Y.

X        A    T    G     C
P(X|A)   0   2/3  1/3    0    - С вообще не будет выбран, а T преобладает
P(X|T)  1/3   0   2/3    0    - С вообще не будет выбран, а G преобладает
P(X|G)  1/3  1/3   0    1/3   - равновесность
P(X|C)  1/3  1/3  1/3    0    - равновесность
Итоговая, не зависимая от текущего:
P(X)    1/4  1/3  1/3   1/12  - ну всё видно
равновесность - только в двух случаях из четырёх, два остальных - даёт разные вероятности для всех трёх вариантов выбора. Итоговая тоже не равновероятна.

Ну и для подтверждения второго кода, приведённого мной, что там всё так и есть:
X        A    T    G     C
P(X|A)   0   1/2  1/4   1/4   - преобладание одного
P(X|T)  1/4   0   1/2   1/4   - преобладание одного
P(X|G)  1/4  1/4   0    1/2   - преобладание одного
P(X|C)  1/2  1/4  1/4    0    - преобладание одного
Итоговая, не зависимая от текущего:
P(X)    1/4  1/4  1/4   1/4   - равновесность(хм)

Который вариант лучше, это ещё вопрос, при условии что я не накосячил.
I see an issue in both subject and first post -- both values should depend on the message that the link points to -- otherwise module will display wrong text in almost every row (first post in the topic).
For the record, I was dumb. I should have said `((1 << flag) == ROOM_GODROOM)`
what's this part for?
I actually like the void pointer that was here before – it's a better way to ensure uniqueness, and these keys never get logged as anything other than pointers anyways.
May want to change these to `robot.reply` as they seem rather generic, and may trigger in general conversation.
does this change to `kpvcrlib/crengine` imply we are switching to a newer crengine version?
Yes, we need a more elegant way. I will look into it this weekend. Feel free to change the UI framework if you come up with any good idea.
Have a look at the documentation above the method declaration in the implementation. That is the documented API and has been for a long time. Can you stick to that API and add docs for your :read addition?
这里似乎有并发隐患，假设以下场景：  
1.第一个线程看到this-handlers为空，进入第一个swap!，然后进入第二个swap!将handler加入集合，这时候this-handlers至少有一个handler。
2.第二个线程也是看到this-handlers为空，也将进入第一个swap!，将第一个线程的handler置换为空，丢失了第一个线程设置的handler。

是否直接用locking宏更安全一些，这里性能不会是问题。因为注册只有一次一般。
Could you please make `ConfigurationEntry` a separate type (ie. not a nested one)?
For a comment like this that describes what the method does, go ahead and use a [docstring](http://www.python.org/dev/peps/pep-0257/#one-line-docstrings).
but then catch silly values to prevent accidents...
On 08.10.2012, at 23:22, Alexandre Gramfort <notifications@github.com> wrote:

> In mne/fiff/raw.py:
> 
> > @@ -903,6 +904,63 @@ def _update_projector(self):
> >              return False
> >  
> >  
> > +class RawFromMerge(Raw):
> I'd rather have a function for this. I hate objects as you know :)
> 
well, so you mean, copy an existing instance of raw and override its attributes? i thought an alternative constructor would be cleaner.
> can you clarify your use case? I am sure why you want a feature like this.
> 
use case 1 is exemplified in my current ica code: the denoise raw calls this constructor to create a new raw from the ica-cleaned data. it also could be helpful to create raw instances from externally processed data.
does that make sense?
> —
> Reply to this email directly or view it on GitHub.
>
1.9.2, right?  And also put 1.10.0 ?
I can't think of a reason why to use reflection here. Let's change it to normal getter calls.
# Escaping variable name with prefix

I don't get the problem with it.

The "playframework_escape_" prefix only appear in router and reverse router. You may use it just as before:
```
scala> import controllers.routes
import controllers.routes

scala> routes.Application.route("foo")
res0: play.api.mvc.Call = /routes?abstract=foo
```

The only problem with it is if you want to get a route named like:
```
GET /       controllers.MyController.foo(case, playframework_escape_case)
```

This code is to generate following scala in routes_generated.scala:
```scala
// @LINE:61
case (playframework_escape_abstract) if true => Call("GET", Routes.prefix + { Routes.defaultPrefix } + "routes" + queryString(List(Some(implicitly[QueryStringBindable[String]].unbind("case", playframework_escape_abstract)))))
```

See, if we don't escape it, variable name gets named as case(abstract)


# Escaping variable name with `

I tried with following route:
```
GET     /routes                 controllers.Application.route(`case`)
```

And play compile returns me the following error:
```
[error] /Volumes/Work/work/Play21/framework/test/integrationtest/conf/routes:54: Compilation error[`)' expected but ``' found]
```
I'm not sure I understand the point of using prompt_char if you use vcs_info
wrong. It will make both of them use underscores
You need to add a `,` at the end of this line.
Slightly different - the normal one simply prints as `handle()` - so if we need to highlight which class this is on, we need to do this.
不太明白？
You should probably have another test to describe getting a non default value back.
Then we need to rethink this structure.
Settings should be created once, in the MailboxType, then it can be passed into the MEssageQueue (def create)
Why this waste of developer time
I did not change the code that handles the reload_templates logic I simply documented it in index.rst. the logic that was added was for other settings.
It should not be very cumbersome to switch to python-drmaa instead of writing all those bash files before the run... do you think you can call it via the pythonic interface instead?: https://github.com/SciLifeLab/scilifelab/blob/master/scripts/test_drmaa.py
Agree. Please ignore this PR and use it as a reference when you work on cart bundle to fix this.
revert to 5
No doubt about that it generally should work, I just would not like to get an NPE unneccessarily.
"Lord Pepperington the Eighth" I'm not eve sure what that means
Never ending wait... it should have a timeout and should be handled.
Vladimir, I was looking at the exception in the JIRA description, but I mistook the java.lang.Exceptions for ExecutionExceptions as well:

Caused by: java.lang.Exception: Failover execution failed
Caused by: java.util.concurrent.ExecutionException: Failover execution failed
Caused by: java.lang.Exception: Failover execution failed
Caused by: java.util.concurrent.ExecutionException: Failover execution failed
Caused by: java.lang.Exception: Failover execution failed
Caused by: java.util.concurrent.ExecutionException: org.infinispan.CacheException: java.lang.RuntimeException: Failure to marshal argument(s)
Caused by: org.infinispan.CacheException: java.lang.RuntimeException: Failure to marshal argument(s)
Caused by: java.lang.RuntimeException: Failure to marshal argument(s)
Caused by: org.infinispan.marshall.NotSerializableException: org.infinispan.CacheImpl
Caused by: an exception which occurred:

In this situation I think we should throw/return the CacheException. If you went all the way down to the root cause, you'd get a org.jboss.marshalling.TraceInformation, which wouldn't make any sense for the user.

If it's not possible to filter out only the java.lang.Exception and the java.util.concurrent.ExecutionException lines, then I think it would be better to leave the exception as it is now.
I've fixed the log.isInfoEnabled().  The instanceof is unavoidable since it is the cause of the ExecutionException that I am checking on.
Why are you changing the dependency?
Need to prefix this with `self.selenium.` and for the locator you need to reference the one on line 69 above so remove `self.` . This will make it reference the local variable rather than the class variable!
So i'll split the string this way roccivic.
Thanks.
Can you clear up what a "Clojure matrix representation" is? Sequence of sequences/
what are these functions doing here? if you want to leave debug code in then please do the timing using CTime
Желательно чуть больше придерживаться лимита в 80 столбцов, хотя бы приблизительно. Таки не очень удобно такие строки переваривать..
Yes, as a matter of fact there is. Stay tuned.
LOL. that was a reminder for myself at a stopping point once. Fixed in the PR upstream
here this almost implies this class should be inherited from a WSF_CONTEXT_HANDLER
which might not be always the case .. even if this might be 99% of the cases.
Please put the error code at the end: `if (sigaction(...) == -1) {`
Are you sure ``$this->returnValue($bundles)`` has the correct type as "object<PHPUnit_Framework_MockObject_Stub_Return>" is not a sub-type of "object<PHPUnit_Framework_MockObject_Stub>"?
The fact that there is no workaround needed on other platforms pretty much guarantees that it's a bug.

The spec for events is also part of apidoc. http://docs.appcelerator.com/titanium/2.1/#!/api/Titanium.UI.TableView-event-click

index is documented to be a Number, so if it's of type string, it is wrong
Sorry to be picky, could you please fix spelling in this comment (exist is misspelled exsist) and add a space after a # :) thx.
Can you add a test for what happens when the only post version(s) is/are prerelease?
No need for the extra `options` check here since `options` is defaulted to `{}` above.
Which is adjustable, you can simply modify the requirement from the package.ini file.
Wouldn't it be better to add an `if (obj.hasOwnProperty(key)){ ... }` around this? Copying all the attrs from the object's prototype probably isn't what you want, right?
It should work in our case as well, but it seems to me it can be useful to have both options (`--isolate` and `--single`). Because `--isolate` assumed to produce one additional process, but `--single` option are not. And I think both scenarios can be useful for people.

I understand that you want to keep it as simple as possible. But that give people a kind of freedom to tune the tool to fit their needs better. People (mostly) do not use option often, likely they create convenient aliases for shell scripts which do the right thing. That's why it seems to me reasonable to leave that option.

What do you think?
Enig, tar den i bruk.

On Sun, Sep 23, 2012 at 12:14 AM, Jari Bakken <notifications@github.com>wrote:

> In puppet/modules/munin/manifests/master.pp:
>
> > @@ -0,0 +1,57 @@
> > +class munin::master(
> > +    $package_name = $munin::params::server_package_name,
> > +    $common_package = $munin::params::common_package_name,
> > +    $package_ensure = "present") inherits munin::params {
>
> $package_ensure-variabelen ser ikke ut til å være i bruk. Den bør brukes
> eller fjernes.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/holderdeord/hdo-puppet/pull/1/files#r1668269>.
>
>
Tror ikke denne docrooten vil fungere med mindre `munin::server` kjører på samme node som applikasjonen (`hdo::backend`, som sikkert burde renames til `hdo::app`). På sikt ser jeg for meg flere app-servere og kun én `munin::server`. Kanskje teste det ut ved å kjøre opp kun `munin::server` i Vagrant en gang.

Er jeg helt på bærtur, eller bør docroot her være `/var/cache/munin/www`?
Great, thanks @eddieajau
Done. Should be ready to merge now.
actualziada
Because this is then messed up.
I like the idea of avatars!
But none == not required. So the question is if want this field to be required or not.
We often name just `user`. We mostly know that it is an object. ;-)
No. You should always use it according to API. API says you pass the number in. If it does not work, somewhere else is a problem. So use it as it is expected, and then we can maybe debug it (or you can open a ticket/TODO for somebody else to debug it).
No, anonymous functions should have space after `function` part. [See our style example](https://github.com/wlanslovenija/PiplMesh/pull/199#commitcomment-1707793).
Also needs:

    other_args = []
I think rather than this the doctor check should just return if the system is Xcode without CLT.
why the onscroll event? can't we use the timeout as per devices that don't support native position fixed?
Yes, I add an issue to manage the "count/len" optimization: https://github.com/liberation/redis-limpyd/issues/25
misaligned. fat finger?
what was the motivation for this one? low probs don't contribute much anyway? what values do you expect to call with?
This shouldn't be needed at all, the logic in build.xml should take care of this.
I'm not sure about the "name", seems it's only used for settings and to display on the subtabs and temporary tabs, and none of that applies to your views? Maybe it should be empty string then, "Detail" too. They are also very generic and could collide when they do get settings.
Yeah, that's a bug. not sure off the top my head what the correct values should be.
What do you mean double assignment? I just vectorized it, so you compute both at the same time and then reuse mu in the sigma calculation. Before this was computed 4 times instead of 1. The [::2] notation just does this

```
>>> x = np.arange(1,11)
>>> x
array([ 1,  2,  3,  4,  5,  6,  7,  8,  9, 10])
>>> x[::2]
array([1, 3, 5, 7, 9])
```

This is because of the (somewhat odd) order of nuis_params, but I didn't want to deal with changing this.
does this restriction matrix have an interpretation/meaning or is it just random?
Rename to onComplete
"scaffold" seems like a strange name for this ...
Yup, you're right. It still has to be in `ddply()`, though, since the count is separate for each group. Also, is that an extra comma in your example?
view -> viewed (I have a feeling that Rachel is taking care of all the typos, and more, right now...)
This is an inherited error (from me I believe), but I think there should be an extra ` after the exp[***] expresssion.
Before merging my changes, you should change the version information to the right values here. I did not find any version information and therefore simply used any values. Of course, the whole version information could be removed (also from the config module), but I do not think that it is a good idea to have a library without a version.
Updated to return NOT_FOUND - the client should decide if deleting a missing resource is the same thing as deleting an existing one.
Yeah, probably. Good catch!
sure about this change? the other coffee files there are not needed?
I refactored the code. Is it better like this?
initialize work
nit: you don't need the () around this.updateMsgTextHeight
I prefer this type of logic to be in cfgstats itself. The engine should not know anything beyond this is a CSV table. You make to many assumptions with this code.
Can't we just do this:

```
$query = (array) $query;

if ($query[0] == true || $query[0] == false ) {
     return true;
}
```
This may be unnecessary since the edge transition on the socket already told us that it's ready.
This function is giving a trajectory to the waypoint. Any objections to calling it pathplanner_simple? Or pathplanner_rudimentary?
Hi Daniel, just for my understanding why this comment was removed?
Wow! The Pagodabox team were pushing me to use it for a PHP 5.3 site I was running and it worked great - I had no idea it was discontinued. 

Ok scratch that then.
Вот эта вся конструция не очень читается. 

```javascript
var Table = function () {

};

var Form = function () {

};
```
@ctweney done.
add the suffix _locator
What do you mean by 'accepted in a previous submit'? Is every test run not isolated? If not, it should be.
カタカナに修正します。
or are you talking about the "#pragma once". I've now removed this.
I still question the reasoning for putting the attributes in metadata.rb when they are never even used.
Duplication of code causes for duplication of data, and therefore potentially divergence.
I believe we can actually change the `:url` to be `:url => 'https://papertrailapp.com/api/v1'` to cut down on the typing and not need to define the `@api_prefix`. It's one of the things I really like about faraday.
This should be OK on Windows.
Wikipedia gives:

- Scharr, Hanno, 2000, Dissertation (in German), Optimal Operators in Digital Image Processing .
- B. Jähne, H. Scharr, and S. Körkel. Principles of filter design. In Handbook of Computer Vision and Applications. Academic Press, 1999.

English, freely available online would be ideal.
Also, I think the three plots should be scaled the same way.
I guess that would be better...
I will also need it for enet_path, the n+1 step on the path is warm-started with the coefs and the ever_active_set from the nth. But that's not implemented yet, that's the next step I want to do. I think we better discuss this when I have working code for the enet_path case too, before someone start committing to my brain :P
I agree. I was working on this but then I ran into the problem I mentioned on the ML. I'll fix that now.
Can you use `safe_sparse_dot` here?
Since we intend to redistribute the file, I think it's better if there's an explicit license.
you can probably initialise these three variables on the same line, using `[]` rather than `list`
I don't see where this code would be doubled / which methods return the same... `toString()` should not be used for code that matters - its more like 'oh - just write it to STDOUT or logfile' in my view.
maybe use a relative path here or one rooted at GOROOT
Well, sure. Although I've never ever been interested in InterruptedException from Thread.sleep, always found those annoying. That method can be inlined in waitForCondition instead and catch ( InterruptedException ) could throw... something, maybe Condition#failure()?
We need to figure out how can native look reused.
I noticed that this morning.  I was thinking '123,456' would come back out as '123456'.  Looks good to me.  I withdraw my concerns.
It will be there whether or not you see it, but I removed the visible implementation :)
? why if packed will the rows be kept in memory?
Extreme nitpicking but, it's not "source", it's "data". E.g. this works:

```python
from django import forms
class MyForm(forms.Form):
    name = forms.CharField()
f = MyForm(data={})
```
I prefer to have an CONTRIBUTORS file, which I think github can even automate that. So I don't like all these copyright, notices on top of code basically.
nit: you may use if/else statement as you do not intent to return a value from this one-liner
Added comments to clarify.
The break 2; drops us out of the foreach as a result:

// Does NOT work:

            'Zend\Db\Adapter\Adapter' => array(
                'parameters' => array(
                    'driver' => array(
                        'driver' => 'Pdo',
                        'dsn'   => 'mysql:dbname=zf2tutorial;hostname=localhost',
                        'username' => 'rob',
                        'password' => '123456',
                    ),
                )
            ),



// DOES work:

            'Zend\Db\Adapter\Adapter' => array(
                'parameters' => array(
                    'driver' => array(
                        'driver' => 'Pdo',
                        'username' => 'rob',
                        'password' => '123456',
                        'dsn'   => 'mysql:dbname=zf2tutorial;hostname=localhost',
                    ),
                )
            ),
just saw that you just changed indentation, but you could improve the code style ;-)
nah, my test implementation assumes an iterable object so None breaks it. I suppose it should handle both None and empty list to be more robust
nitpick - casing
that's specific to my project, i'm gonna edit that
Exceptions are Serializable, and serialization requires this. Or at least Eclipse gives warnings when this isn't here. See http://www.javablogging.com/what-is-serialversionuid/ for an explanation of what this is for.
please remove methods using the camelcase.
@shacker This is more parsing of the feed that doing the actual rendering.
Can we create a separe function for this and pipe it through after the Ajax request?

e.g. parseCampusNews
"fpr" -> "for"
Cool..
Can you please set `@model` on the same line you call `define_model`, like you do elsewhere?
TODO add com_content as default but where?
I think I actually fixed this in the terrain branch.  I believe the best fix is to change `CompositePrimitive` to initialize `_centralBody` to `undefined`, not `null`.  That code is very old.  Can you make that change here so we don't break when the terrain branch is merged?
The material texture cache is not responsible for deleting textures, it merely stores the same reference to the texture as the material itself. As a result, if a texture that belongs to a material is destroyed, every other material that has a texture with the same image path will also be destroyed.
There are multiply set methods that trigger `recreateBillboards`, it seems really bad performance-wise to recreate them multiple types if I change multiple properties in a row.  Maybe in practice it's not that big of a deal.
I will change this to 0.6.1
This value can be set as part of configuration (inside actual cfg file with latest update)
Oops. Fixing
If the user uses view.fireEvent('someEvent', null) to fire a custom event, data will be "null".
you could use: `bean`, so that the test would be re-written as: 

```clj
(fact "make-query"
    (bean (make-query "test" 10)) => {:page 10, :rpp 100, :query "#test"})
```
This makes of course sense, I believe there's another pull request for this issue, will look into that.
This method will be hidden by the following
да, не надо триггерить с неймспейсами, это как-то совсем против странной семантики jquery
This should emulate what `ElementType` is doing and result in `void` if `T` isn't a multidimensional, static array. It makes no sense to get a valid result for invalid input as this is doing now.
How about changing:
!(!isNarrowString!R1 && !isNarrowString!R2)

(isNarrowString!R1 || isNarrowString!R2)

double negation is kind of hard on eyes.
It looks like it works, but it turns it into `std.range.takeExactly(r1, n)` and makes the whole thing a link, which is kind of ugly IMHO. I can certainly make the change though if you really want me to.
nit: space between for and open paren
If this is ok with you, I'd prefer that. Thanks :) We can also change it later on in case it doesn't work out that good.
Only debug output, and it's useful to add the `...` imho, it indicates that what follows is the actual action being performed.
It should...I wish `supports` attribute would work with platform_family! :)
this is wrong. You need to add the leading ``\`` if you want to use a FQCN. Otherwise, you should use the relative class name (relying on use statements)
What's this for? Why include this?
nice!  note: this test does not require the as_json method.
Yup; I was thinking of both, actually. In my own CLI scripts, I'd do flush and clear every X iterations as part of a batch. Clearing is more useful IMO, as it lets you reclaim memory. In other cases, I may never want to flush, as I don't want to modify anything through UoW.

I can't think of any use case offhand, but if I implemented something with this I'd likely want to do my own flush in the process method.
oh, this isnt the fix I want, this assumes that your patches have been merged. I wanted something that just sets the region param.
is there no way to explicitly reload the cookie other than deleting all of the data?
FWIW I agree with @mmoss for the long term, and would prefer if we switch to being consistent with bootstrap immediately if possible. @m5o's solution makes me scared of unforeseen consequences if we go with it, and the existing 0-1 range makes me fear mistakes made due to inconsistency.

I'm still not sure how to solve the compass issue...
Почему это?
Лучше юзать `isEmpty ()`.
The keyword already contains a space, so son't add an extra one.
For Spree 1.0, this should be spree/users/show.
@alexrp: Think about the implications concerning `@safe`-ty – you can read from an arbitrary memory location in safe code via dereferencing a null pointer to an object containing such a humongous array.
Should catch Exception here, not Throwable. Otherwise we may suppress an Error.
BTW, you misread my comment - I thought it was easier to read when the returns were there :-)

Did you fix the other place?


					else {
						theOneIWant = true;
					}
Thinking more. . . what this one meant is to protect from multiple @Payload annotations (with xml its not possible), so I guess i am going to need another boolean to be able to distinguish how payload was set XML or Annotation and if Annotation then throw exception but if XML then don't even bother checking Annotation
The above are correct for the develop branch, but not the master branch.
не хватает `
`return last 10 blog posts` -> `return the last 10 blog posts`
`listing last 10 blog posts` -> `listing the last 10 blog posts`
全体的に書き方を ResourceSuite に合わせて下さい。
Please use underscore_separated for configuration files, so domain_name
this one will be required to handle composite keys
there should be an upper bound