labofsahil/github-event-dataset-2012 huggingface dataset:
POSTs for configuration are done to `/api/config` now just as the GETs and context is derived from there.
Shouldn't jake just be a devDependency?
我是觉得 `validateReturn` 的职责过多了，把样式处理单独提取出来有利于加入新的验证，比如说 `<select multiple=multiple>` 的验证。

这些代码再想想吧，提交 pull request 是一起 review。

如果需要写一个更好的 `validateReturn`，这些代码也要移出去别的地方

```
message = message ? message :
  pattern ? (new RegExp(pattern).test(val) || 'unvalid') :
  patterns[type](val) || 'unvalid';
```
skstream is a socket library. Do you really need this?
Put spaces around the exclamation mark (see the [coding style guidelines](http://codeigniter.com/user_guide/general/styleguide.html)).

Also, spacing on this and the lines below looks weird.
Есть также тип NSUInteger (unsigned integer) - он в этом случае подойдёт больше.
Can we please change this filename to csslint_node.js to be consistent with the naming scheme used with jshint?
I was coincidentally thinking about doing this yesterday. Good move
Aaah, this is what was confusing me, GenericCustomArmor is sent directly server-side, while the client has a proxy PacketCustomArmor class... it *works*, but it's not really a good design :P
I guess what I am trying to get at is that if this is a temporary measure then this seems too much like a permanent solution to make me comfortable; temporary solutions should, by my lights, introduce enough pain to require the switch to a permanent solution rather than alowing a sub standard one to exist any longer that it absolutely must.

I would advocate changing the static `SUFeedURL` to point at the `changelog_beta.xml` file instead for now, so that we are forced to adopt the right solution before a stable release happens.
the exception message is wrong. The effect is not about grayscaling
NO idea... its been tehre for awhile... lets see what happens if i delete it
This is empty.
Nit: comment should start with a capital letter, and should be 80 char or less (if longer, just break onto the next line)
Doesn't this fail to calculate the correct argIndex in a case like this?

    [obj rac_liftSelector:@selector(setAnObject:andAnotherObject:) withObjects:@"constant object", aSubsribable];

This look like will make `argIndex` equal 2 when it should be 3.
I changed all three logger calls in this function.
Mention that None is regular grid conn. (as above)
Member. So it reads "for member in collection". It's kind of like using i for iteration.

I'll switch it to "clause" here--that'd be clearer than the general "mem".
Why the strange name here?   Was this originally written by Daniel in the branch but never in master?  If it is in master somewhere, where is that deleted code?
I don't think we should allow for this. It's very rare that you should be getting all data, possibly thousands or milions of rows ;-) IMO we should have something like List<T> getAll(boolean includeVoided, Long firstResult, Long maxResults) and also Long getAllCount(boolean includeVoided).
I think combinations should be unique even amongst retired ones
change to Boolean
not needed when bin\kuduSync.js exists.
I don't think this is a usefull note, people can choice themselves which syntax they like and if you want to add this note somewhere you should add in in the `book/translation` article, it is irrelevant here.
加入一段内容=>添加一段文字
它 => nginx
内容=>文字
You are skipping 16-bits here. Why?
This is to clean up the links from google news feed, since they pass through google first.

Example:

    http://news.google.com/news/url?sa=t&amp;fd=R&amp;usg=AFQjCNGGHMAa4VotXSSr4TcYfcLp5Ki_JQ&amp;url=http://www.nytimes.com/2012/11/06/world/middleeast/Syria.html?pagewanted%3Dall

We want to strip off the "http://news.google.com/url?" part.

The more general approach which I used was to give the WebSource model a new field, called "url_processor". This processor is a regex string that looks like:

    s/{find}/{replace}/{regex flags}

Replacing everything inside `{` and `}`

Theoretically, we could use other processors besides "s" for "substitute", but I'm not holding my breath. In the meantime, it allows us to specify how to process the detail urls scraped from a given WebSource (such as google news).

NOTE: You can see the fixtures above for an example field value for "url_processor".
Why should leading underscores not be used?
ANTS -> ants (also in the other doctests)
Ты мне предлагаешь запилить на каждый кастомный команд по функции?
Для такого грех не пользоваться uniform initialization syntax.
Aren't this needed?
I'm a little confused my this statement. If you are talking about Ruby's call chain, then yes the instance of a class, or object is at the bottom of the chain and the first place you look for a receiver of a message (method). The methods defined on a class are available to all subclasses unless they are overridden. You can have what is called a singleton method defined for a particular instance (object) of a class, and that could override what is in the class for that object, but I'm not sure if that's what you are referring to here.
s/int/integer/
The name can not be workflow02
please rename to `checkBaskets`
this kind of static use must be discouraged
no extra : notation "if( (a=b) )"  is a good practice to enforce a test containing affectation, "if(a=b)" issues a warning with lint or c.
better : "if( (a=b) != false)" but seems buggy in php if b is simplexml object (?)
not tested "if(!!(a=b))"
`null !==` comparison can be removed
The sections rendering strangely are the ones that are missing
$base-rhythm-unit was only ever used in this one spot, and has been removed.
How is it that ```except``` can always be assumed to be an Enumerable?  What if someone passes in a single symbol (as might be assumed to be valid based on the ```fields_for_spec``` tests)?  I think it makes sense to leave it open to receiving either a symbol or an array of symbols, as is done here.
I like @jcruz7 way of doing it better. Is there any reason you want to replace the spaces first before splitting?
How about moving this above the init method?
Yeah, I was thinking about embedded mode, but I guess that is out of scope.
What's this one do? after updating it i'm getting a **Schema validation failed for request**, changed that mesage to output the whole response with response.inspect and i see 

    {:fault=>{:detail=>{:fault=>{:reason=>"Schema validation failed for request.", :details=>{:validation_failure_detail=>{:message=>"Expected element 'LabelFormatType@http://fedex.com/ws/ship/v10' instead of 'LabelStockType@http://fedex.com/ws/ship/v10' here in element LabelSpecification@http://fedex.com/ws/ship/v10", :xml_location=>{:label_stock_type=>"PAPER_LETTER"}}}, :error_code=>"5"}}, :faultcode=>"soapenv:Server", :faultstring=>"Schema validation failed for request."}}

If i remove the label_stock_type it works again, i'm calling the label creation with no custom format and it used to work.
why not simply using require_once instead of using a constant ?
if it's not necessary, I don't know if it's relevant to set it to true by default.
Ohhh… yeah fair shout.
I don't think click() works for keyboard?
That is one thing, but the main advantage is extensibility I think. Doing it this way kind of allows you to pass an object to an extended interface and basic implementations will just cast it instead of choking on it.
"No new newline" sollte das wohl heißen ;)
Let's just delete this trap instead of commenting it out
So it's not working b/c the call to [`positions`](https://github.com/reddmetrics/forma-clj/blob/d8414e29da3a6c07b91a9e225d028145d7ce97fc/src/clj/forma/utils.clj#L79) uses the behavior of `=` instead of `==`.

I'm going to keep the behavior as is. If you're modifying a timeseries, it's your responsibility to make sure you're doing it correctly. And since the type comes from the underlying data, you should probably tailor your use of `replace-from-left` to the specific data you're using, rather than depending on some assumption within `replace-from-left`.
Yeah, not sure how I missed that
Oh man, I can't wait to get rid of stuff like this..
@arnodeclercq Can you make the themes variable local to this function?

This line should also comply with our style guide.
I'm still of the opinion that a switch is not a good UI solution for this problem, see [open.ss.org commet](http://open.silverstripe.org/ticket/7412#comment:30). 

@clarkepaul Could you give some feedback on the open.ss.org ticket, particularly how you would see the "future state" UI implemented?
```ruby
browser.alert.should exist
```
Why write this code this way, why not use the direct comparison of the values?
Fix fully qualified namespace.
As jquery code style guidelines days "===" should be used in favor of "==" so I am used to make any comparison in JS with "===" whenever possible, I just forgot to keep the pattern, I'll fix that also.
@thradec @tombentley  No, it's not supported.
I think its missing content promise. edit_line or something.
there is already a Text extension. Do you really need to create a String one ?
nvm i forgot that we have a method called .redirects
There is no reason for `/` be excluded. I am using in some other projects the following configuration and it seems to work nicely:

```
# URL prefix for django-pushserver passthrough callbacks
PUSH_SERVER_URL = '/passthrough/'

PUSH_SERVER = {
    'port': 8001,
    'address': '127.0.0.1',
    'store': {
        'type': 'memory',
        'min_messages': 0,
        'max_messages': 100,
        'message_timeout': 10,
    },
    'locations': (
        {
            'type': 'subscriber',
            'url': r'/updates/(.+)/',
            'polling': 'long',
            'create_on_get': True,
            'allow_origin': 'http://127.0.0.1:8000',
            'allow_credentials': True,
            'passthrough': 'http://127.0.0.1:8000' + PUSH_SERVER_URL,
        },
        {
            'type': 'publisher',
            'url': r'/send-update/(.+)/',
        },
    ),
}
```
Why exactly you have to push this as a JSON?
(The import here is of course correct. Don't change anything.)
Rename into `callback`, not `oauth`.
You could just define default value here.
Try simply putting `obj` in there. ;-) Maybe JSON serializer will manage to serialize it.

But this is not good approach as some additional data (otherwise not accessible through API) could be leaked. I would do internal request to our API (not through HTTP, but directly through Python) and request this post as JSON, and include it here. ;-)
Is not this profile page? So `profile.html`?
Need to abstract out this a little, as it's now duplicated in Cohttp_lwt too, and is needed by pretty much all the protocol implementations we plan to functorise.  I guess it fits in the eventual actor library...
No need to include the module again if the superclass already has it
This should be correct.  If any thread in our ThreadGroup dies I want to
free the lock manager.  We only register after creating the LM, so it's
definitely there to clean up when we crash.

What behavior did you have in mind?
On Nov 12, 2012 1:06 AM, "Moxie Marlinspike" <notifications@github.com>
wrote:

> In src/org/thoughtcrime/redphone/RedPhoneService.java:
>
> > @@ -188,6 +189,16 @@ private void initializeResources() {
> >      this.lockManager      = new LockManager(this);
> >    }
> >
> > +  private void registerUncaughtExceptionHandler() {
> > +    Thread.currentThread().setDefaultUncaughtExceptionHandler(new Thread.UncaughtExceptionHandler() {
> > +      @Override
> > +      public void uncaughtException(Thread thread, Throwable throwable) {
> > +        Log.d(TAG, "Uncaught exception - releasing proximity lock", throwable);
> > +        lockManager.updatePhoneState(LockManager.PhoneState.IDLE);
> > +      }
> > +    });
> > +  }
> > +
>
> setDefaultUncaughtExceptionHandler is a static method that, I believe,
> applies to all threads. This appears to be written as if it expects it to
> only apply to the local thread.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/WhisperSystems/RedPhone/pull/60/files#r2094363>.
>
>
That also makes sense for me now. It is a lot smarter to present this as redirecting you instead of requesting the user to confirm it again.
Sigur e ok if-ul de aici? Parcă adresa trebuia să apară tot timpul, și numele organizației să apară doar dacă omul reprezintă vreo organizație.
I thought this was already in. But yes, we need to fix that bug for all supported platforms.
> We don't allow {a: a, a: b} because of duplicate keys

I think this is why I wanted the sanity check;  LHS destructured object literals assign to the vars named for their *values*:

```coffeescript
(function(_arg) {
  var a, b;
  a = _arg.a, b = _arg.a;
});
```
> It's ambiguous with a series of destructuring parameters

I totally agree that it can be abused, but I'm fairly certain I've seen it in the wild, and elegantly executed. I'll have to try to remember where.
initalised -> initialized
So far the plugin is only used by the core, it's the only part with thrift and protobuf files.
Split these comments in two.

Something along the lines of:
    // Either an array of observers of this promise (if the promise is not fulfilled yet)
    // or `null` (once the promise is fulfilled).
    var pending = [];
I don't particularly mind that one, I was just wondering if it was a typo.
I went this path, because not all features are user controlled. ``status``, ``is_shared`` and ``is_forkable`` are, but not ``is_removed`` and ``is_featured``.

I am inclined to keep the status as constants since they allow me to validate the values quickly (one query less), are coupled with the application behavior 
and are very unlikely to change.

I think I can incorporate ``is_forkable`` and ``is_shared`` as an status, since:

- In order to share it should be published and 
- In order to be forkable it needs to be published, and probably shared.

But I'd need to run this through my team too.

What do you think?
Will fix the alignment.
`parent_view.handle(action_id, event, target);` is even better.
UNREACHABLE()?

(Though I noticed that mod_pagespeed doesn't have that macro.)
Yeah, it will assert. Probably bad. What I could do is add a "unknown" icon of a box with a cross in it and return that if its not found. This is more or less what the texture loader does - if the file isn't found, it uses `data/textures/unknown.png`.
Solved the issue of just dumping it the subj array, but just realized this makes it break on unicode names (eg: bookid 19238).  Looking for /proper/ way to do this
I doubt anyone's using this, but I'll change it.
"gets sigma, checking that everything is ok and handling dimensions" ;)
Great idea, will change, thanks!
That's clearer.  Thank you.
I found x confusing, since we also have (x,y) positions, so I thought r being the radius in units of r_s was clearer.  But if you think it's clearer the other way, I don't mind if you change it back.
I forgot to highlight this line.  I'd like to leave nproc = 5 for this script to point out that it works, but I don't particularly want to put all the multi-processing stuff into demo10.py.  We have it once in demo9.py, and even there I'm not sure how useful it is as far as the tutorial goes.  I'm worried that it makes it harder to get to the GalSim stuff that the tutorial is really about.

So what do people think?  Is it ok to have demo10.py be single-process and demo10.yaml be multi-process?  Or should I remove this line here and in demo10.json before merging?
The keyword "obs" is not very illuminating - could be "obscuration"?  (admittedly a mouthful, but very clear what it's for)
Should put a line at the top of the file:
```
//#define USE_COS_SIN
```
with a comment about what happens if you uncomment it and why you think leaving it uncommented is better.  (Assuming you do.)

Of if this is just an old bit of code that you now roundly reject, you should either remove it or surround it with `#if 0` instead and a comment that the other branch is better.
Should be @ instead of $
Just delete this class, and update refs to the new location. It hasn't been in the code-base for that long...
`this.url = checkNotNull(url, "url");`
This one, with an array of 300000 and chunks of 13 took 18ms.
Can't you just use the shared_ptr straight here?
Agreed. I'll remove both of these.
I don't think adding it to MongoLog makes sense, but we could add it to utils.inc
The environment is modified by adding "Requires" to the containment. It works as follows:

1- ibs will create a containment where to run the kiwi job that creates the containment image
2- ibs will install containment-rpm in that image, and will do so with the containment-rpm runtime requirements. Containment-rpm installs the kiwi_post_run script.
3- ibs will call kiwi inside containment
4- ibs will call the kiwi_post_run hook (that will call your script)
5- our kiwi_post_run script will create the RPM based on the image spec file.

Thus, if you want ruby to be available in the environment were kiwi_post_run is called (and your script), it needs to be a containment-rpm requirement.
Isn't `-ms-transition` an IE10 early preview thing only? I thought IE9 didn't support it. Same issue for `#btn_resize`
Tests that use the django test client are functional tests and should be in tests/functional.

I've fixed this.
Yep, should have done that.. copied from jasmine lib directly and changed the export value.
@stof I don't get it.
I had a fix for this on the pre-merge code that prevented it wrapping things in your inventory, so you could put it in a container or something.  I'm busy as hell right now, but you might want to take a look at it.
make it "cherry flavoring packet", and "berry flavoring packet" respectively.  It will help make the text parsing better.
It's hard to see which ``if`` corresponds to this ``else``. I think it would be better to use braces and indention.
```
if (world.logError) {
    world.logError(e);
} else {
    console.error(msg);
}
```
Any suggestion for a good name?
Integer.valueOf лишний
÷ ÎÏ×ÏÍ ÐÕÌ ÒÅË×ÅÓÔÅ ÆÉËÓÁÎÕÌÏÓØ ×ÒÏÄÅ ÂÙ

2012/11/30 Dmitriy Komanov <notifications@github.com>

> In src/ru/fizteh/fivt/bind/test/Permissions.java:
>
> > @@ -14,6 +14,11 @@
> >
> >      private final Map<String, String> map = new HashMap<>();
> >
> > +    public Permissions() {
>
> é ÏÐÑÔØ ÉÓÐÒÁ×ÌÅÎÉÅ × ÍÏ£Í ÆÁÊÌÅ.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/dkomanov/fizteh-java-task/pull/348/files#r2272598>.
>
>



-- 
ó õ×ÁÖÅÎÉÅÍ
æÅÄÀÎÉÎ ÷ÁÌÅÒÉÊ
íæôé æé÷ô ÇÒ. 195
I'm not sure if I understand this comment completely.
nice catch, I searched the code base and found many other cases like this, fix them all.
i bet this replace needs to be put into a bunch of different projects.
I talked with @Marlena  and @zacc  and I will transform this in a class
pep8 likes it when:
Imports should be grouped in the following order:

    standard library imports
    related third party imports
    local application/library specific imports

You should put a blank line between each group of imports.
the CS should be ``} elseif (...) {``
to comply with xbmcs configure flags, it has both too
I am not sure: why did you remove these lines. Them seem to be useful to me.
> 
> I think I would rather have a function deterministic_svd or just a function svd with make_deterministic=True or something like that.

Remember this is used for the randomised svd as well as the regular svd. So do we add two new functions? deterministic_randomized_svd sounds pretty bad. And if we add flags, we would have to wrap the svd from scipy.linalg.

And we still would have to replace all the imports in the codebase.
spaces before ``:``.
And then you are merging two identical arrays here?
Just because they are not part of alias, they must be present in the write pattern, AFAIU it.
"servers is" should be "server is"
Would prefer to see an if() here, personally.
Was there also supposed to be a password field here?
You better fix this now!
why did we never seen this before and what does that have to do with windows in suggested on the Jira issue?
Yes, I was being lazy / testing out the code reviewers...

I will fix.
Should the dfsgroup command info in daliadmin usage info be updated?
I think this logic is broken. If ```useINotify``` can only run once (as per this close), it should not be inside a loop. The only thing that need looping here is the old check. Also, if the caller provides an abortsem, the queue will be created and not closed, since it might never get here.

This works because when useINotify returns 1, the logic below will find difference and return, but that opens a lead to people introduce code in between and break the logic.

I propose you move the INotify check before the loop, setting appropriate flags if necessary, and only closing the queue before leaving, either by return or exception.
Run _apt-get update_
Why do we need all these when the `content` block doens't use any tabs or any graphs. 

Can you not [do it like this](https://github.com/mozilla/socorro-crashstats/blob/master/crashstats/crashstats/templates/crashstats/report_list_no_data.html) instead?
Sorry @simong , I missed this when I reviewed your changes, but I had mentioned that you should put a `notify` onto the config resources which would take care of #2. I think it might be equivalent to doing a subscribe, though.

Also, I mentioned that `Service['cassandra']` would require the **exec**, not the configs.

Major review fail on my part there. Sorry again.
Is the C# NamingConvention not to capitalize the first letter in a namespace? helloThere vs. HelloThere.
function/class opening braces (`{`) should be on a new line.
Please use an obvious pattern, we do not have such size constraints!
`if (listeners.indexOf(listener) == -1)`
Wouldn't it still be lazy?
Pour l'instant l'id ne nous sert pas, on ne vas pas le mettre.
Забыл запятую!!
Is there a reason for this number in particular?
Nice -- I didn't know about the "@Scheduled" annotation.
We actually don't need to perform this call since we can use `/primary/` instead
need includes
~~~
    (apply str [(:name author)])
;=> (str (:name author))
~~~

Lisäksi `(:name author)` on jo merkkijono, joten itseasiassa

~~~
    (apply str [(:name author)])
;=> (str (:name author))
;=> (:name author)
~~~

Eli pelkkä `(:name author)` olisi tehnyt saman asian.
Just tired using this method on my setup and it fails msierably. This regkey isn't populated on any of my clients
for this method we can use the find_elements technique which returns a list of all elements for a given locator, then daisy chain a find_element.
We can avoid having the complex locator and fewer of them too (important for maintenance).
Our preferred coding method would be:
`self.selenium.find_elements(*self._questions_list_xpath_locator)[question_number].find_element(By.CSS_SELECTOR, 'h2 > a').text`
We should consider ignoring NLS for all the test plugins ...
remove empty implementations of this method
Thanks, completely forget to add them
Should this be startDefaultRenderLoop?  Or just startRenderLoop?
Just assign to this once.
The rare "already optimized" case is one reason, but not the main one.  The primary reason is that these objects are meant to be mutable data objects, so all of their properties should be settable; validation should not occur in the object itself but rather the items using these objects.  Otherwise, we would have to eliminate all member variables and have getter/setter items so that the center gets recomputed every time.  Even if we try our hardest to "protect" the center property, There are plenty of other ways for the center to be specified incorrectly as well; for example by passing an object literal with minimum/maximum/center properties.  Hopefully my explanation makes sense, I don't feel like I 'm doing a good job explaining it.  Maybe @kring or @shunter can give their opinion.
There is a 1/1000 chance of not having a decimal point in what the server sends us. Can you verify what type of dates the server sends when Seconds and Milliseconds are zero? Might need to do a one-off test to check that. Then make sure you cover that in this test
Please elaborate on this comment:  why you're creating a copy of the view proxies. In this case, it would be b/c of threading and remove() may alter the original children list.
There are a bunch of trailing spaces at the end of this line.
Fixed in next update, following Arthur's recommendation.
Spaces around "<".
How about `dontcreate`, see last commit?
I think adding a 'start' and 'goal' (sounds strange, 'termination' maybe?) node to the relation would be a very useful thing because relying solely on the order of the members is not a safe thing to do. But this really needs to be discussed with the OSM community at large. You might want to bring that to the tagging@ mailinglist.
I'm not 100% sure what the goal is here (it looks like an image is being centered on the background of a div), but is there any other way we can accomplish this?
Basically, embedding the CSS like this is not a good idea, and if we could either use a regular image and just use text-align center or some other approach to where we can place the CSS into the main.css file, that would be best.
I don't think that the .gitignore for core bundles should accomodate all the files generated by different IDE's, so IMO this folder should be removed before commit
Actually checking `isDone` gives the wrong behavior. A job that finishes and fails has both `isDone` and `isFailed` set to true.

The current documented behavior of track() is to only call done() if a job has completed *successfully*.
.toggleClass, аргумент switch
No uncommented code please.
``$noInteraction = $input->getOption('no-interaction')`` will do the same
Why the excessive space here?
`iota` is a special case. IIRC, it's because there were issues when using it with `long` or `ulong` on 32-bit systems, since `size_t` was too small, but I don't remember the details. In either case, the normal thing to do is to use `size_t` for indices (since that's what it is for arrays), and that's what almost all of Phobos does. So, unless a particular function or type needs to use `ulong` for some reason, it should use `size_t`.
@denis-sh: So maybe adding a comment in the source there might be a good idea? ;)
minor duplication grumble grumble
Your docstring still isn't :-p
Then this quickstart is in the wrong branch. It's in master-eap6. It needs to be moved to master-wfk.
Sorry, couldn't figure out COM in Fx35. Hopefully this doesn't mess anything up for 3.5 users.
Unfortunately this line changes the language of the node, we need to find something else to get the English l10n.
Could use a (.*) to capture all the datetime stuff and use Chronic to parse. It might look cleaner than all the (\d+)'s.
``master-dev`` is not valid
Well, the program has some weird "make" call: make 'VELVET_DIR=/path/to/velvet'.
I did not test if it is possible to build Oases when specifying Velvet as a dependency and hence if it is already installed on the system. Also, using build_dependecies may do the job but I did not test it. Actually, that might be nicer but I would need to know how to access the directory of a particular build_dependency, then maybe make_opts (or how it is called) could be an option. Neither do I know if Velvet is definitely required for the usage of Oases, so one might not be interested in having Velvet available on its own.

Best,

Cedric

On 8 Dec 2012, at 15:10, Kenneth Hoste wrote:

> In easybuild/easyconfigs/o/Oases/Oases-0.2.08-goalf-1.1.0-no-OFED.eb:
> 
> > +
> > +name = 'Oases'
> > +version = '0.2.08'
> > +
> > +homepage = 'http://www.ebi.ac.uk/~zerbino/oases/'
> > +description = """De novo transcriptome assembler for very short reads"""
> > +
> > +toolchain = {'name':'goalf','version':'1.1.0-no-OFED'}
> > +toolchainopts = {'optarch':True, 'pic':True}
> > +
> > +velvet = 'Velvet'
> > +velvetver = '1.2.07'
> > +
> > +sources = [
> > +           '%s_%s.tgz' % (name.lower(),version),
> > +           '%s_%s.tgz' % (velvet.lower(), velvetver)
> 
> Why do you need to add the Velvet sources here, while Velvet is also a dependency?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 

Cedric Laczny,
PhD Student
 
UNIVERSITÉ DU LUXEMBOURG
 
LUXEMBOURG CENTRE FOR SYSTEMS BIOMEDICINE
Campus Belval | House of Biomedicine
7, avenue des Hauts-Fourneaux
L-4362 Esch-sur-Alzette
T +352 46 66 44 6398
F +352 46 66 44 6949
cedric.laczny@uni.lu     http://lcsb.uni.lu
 

-----
This message is confidential and may contain privileged information. It is intended for the named recipient only. If you receive it in error please notify me and permanently delete the original message and any copies.
-----
Better: "You can use the second argument to define the condition:"
aw. i was testing.... and it seems that I left it commented
what happens if i have other keys in defaults as well? are they lost or just non-editable? 

my question about the control was meant for when we have a  specific value. is it re-shown when editing?
Вот когда у меня будет столько же опыта, сколько у тебя, тогда и буду так делать :-(
I think it has been completed - Zach indicated to me that he had it working.
Will remove. Thanks.
You are doing GET here? Use `getJSON` jQuery function then.
I think you are looking at wrong piece of code. I don't see any sleep() calls in /start handler that precedes this statement. The exec("find /") is executed with a timeout of 10000, which means that command can take at most 10 seconds, and no more.

function start(response) {
  console.log("Request handler 'start' was called.");

  exec("find /",
    { timeout: 10000, maxBuffer: 20000*1024 },
    function (error, stdout, stderr) {
      response.writeHead(200, {"Content-Type": "text/plain"});
      response.write(stdout);
      response.end();
    });
}
I don't know where this came from. Will remove
is this setting really available here? even if it is "this document" would then be misleading, since mapped super classes are not documents.
Or to be consistent with the context you use in a later test, use 'controller present' and 'controller missing'.
Is the indentation here ok?
I've been putting the _ at the end of the instance variable name... this ensures you don't collide with any private instance variables in Apple's base classes (because Apple always uses _ before the instance variable name)
delete in favor of injecting JSONEntityFactory
Maybe it is better to `return` the `value` from your method if it has a value and otherwise `return this`?!
I used EmbeddedListField and EmbeddedSortedListField to be consistent with mongoengine where you have  ListField and SortedListField.


On 9. apr. 2012, at 13:55, Mitar wrote:

>> +        created.
>> +        """
>> +        
>> +        self.fk_resource = self.to_class()
>> +        
>> +        # Try to hydrate the data provided.
>> +        value = dict_strip_unicode_keys(value)
>> +        self.fk_bundle = Bundle(data=value)
>> +            
>> +        return self.fk_resource.full_hydrate(self.fk_bundle)
>> +
>> +class EmbeddedListField(fields.ToManyField):
>> +    """
>> +    Represents a list of embedded objects. It must be used in conjunction
>> +    with EmbeddedDocumentField.
>> +    Does not allow for manipulation (reordering) of List elements. Use
> 
> Wouldn't better name be then *EmbeddedSetField*? Or are they not unique? Sets are often not ordered, while lists are? No? Yes?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mitar/django-tastypie-mongoengine/pull/1/files#r659744
Ahh, then it was my missunderstood. 
OK. I'll do some improvement then.
What was the reason for this change? Perhaps there could be value in having both overloads?
@caseyyee yes but there is strictly **no gain** is adding css class to already expressive markup.
how does this exactly work?
Why the change up here? Just looks like whitespace :S
This is meant to return an array of items with a predefined structure, so it should really be:

first_name
last_name
name
nickname
email
etc

Please can you return as much of this as is available instead?
does it work in development mode without celery?
Add brackets, please:

```javascript
if (m == 'friends') {
   i_bl = 'friend';
}
```
Why do you exclude `E_STRICT` errors? Whatsmore, you shoud start the line with `@` to ensure it throws no error in case script is not able to set this directive.
Вообще пушка.
These are defined by us. Not by native.
For some reason this check was in there as a warning--but we can remove. It's just debug.
I committed a bunch of changes as part of the 1.0.2 release that should hopefully fix the GWT issues, both for compiling and in Eclipse.
Consider moving the table template to main.html