labofsahil/github-event-dataset-2012 huggingface dataset:
So a .ss-ui-action-tabset in tree view, for instance, will be controlled by 4 different entwine calls? Tabset, generic action tabs, actions row action tabs, and tree view?
I guess it could be an unapply, but it should rather return an Either. then use fold in line 176. 

The match ```Some(MIMEType(major, minor, params))``` would use the ```apply```, then ```unapply``` of the case class.
I think __wt_evict_page could be static to bt_evict.c
should be just  WSF_URI_HANDLER
Is it possible to add a ``@param`` annotation for parameter ``$modelData`` which defines the array type; using ``array<SomeType>``, or ``SomeType[]``? Please also check line 482.
Is it possible to add a ``@param`` annotation for parameter ``$options`` which defines a more specific range of types; something like ``string|array``, or ``null|MyObject``?
These changes don't seem to do anything different from the original version.  

isBlackberry is not used anywhere and background is set to fff for all
the help of the command should be added too
Correct.  When you reach EOF the connection is not closed, just like a normal file handle is not closed when you reach EOF.
I think we negotiated with Rouven to remove JError in 13.1 on the condition we could put it in the legacy tree.
It was missing the .php so it screwed up the include :P
Small nitpick, could you move to the assignment out of the function call? I think that'd be more readable. There are more of these in other lines below.
Ditto on since tag change.
Para o seguinte
сюда тоже вставь, плз, заголовок с лицензией и vim modeline
And why mixing GET and POST? Why not have next URL as part of POST?
This is OK and should be removed. This is not used anymore translations.
Any particular reason that you are removing those lines? For different platforms we have observed different files so this is why all this is here. And I don't think there are minuses to having this here. Why then removing?
*from* instead of *with*?
You should not use hard-coded URLs.
Or *userlist*? *onlineusers*? :-)
Empty line after `"""`.
We are doing this differently. Please merge with main repository.
If I create dict here, I can simply save it to database on server side.
Why list? Why not tuple? Will you dynamically change values of dependencies? I think not. ;-) So tuple is perfect for this!
Because I originally thought that maybe we'd want 9 to allow 9 digit zip codes, but that would require 10 characters to store the dash. requiring 5 is simpler than adding a migration, and it will satisfy the vast majority of users anyway.
Good call, but this is in an open source module. I believe we can override this route in Phantom to go to our preferred login page.
I expected this... well this wont be necessary after my improvements. Just wait a bit :)
Use a Map, not a Hashtable
Do these changes have no affect? If so, I'd be happy just making it STATICFILES_DIRS = (,)
I wasn't really sure when to make things atomic. I'll go back through and check them; I doubt most of them need to be
Removing the ```ẁith``` part from the ```render``` tag does raise an error, doesn't it?
`mixed` instead of `string`?
I can understand the desire to not want to override an explicit SessionBuilder.autoclose value.  But the idea of passing in autoclose should really have been this concept of SessionOwner I think.
What's the benefit of this? It looks fine I'm mainly curious.
End in a period for consistency.
According to the documentation it should only return `None` if `stdout` is not set to `subprocess.PIPE`:

> Note that if you want to send data to the process’s stdin, you need to create the Popen object with stdin=PIPE. Similarly, to get anything other than None in the result tuple, you need to give stdout=PIPE and/or stderr=PIPE too.

So getting `None` back from `communicate` shouldn't happen.
Sorry drop this, I forgot we moved watcher_defaults to a function so we're fine
The double-checked locking pattern won't work with a dictionary.
Ahh, my mistake. Forgot that the date referred to the service REST version, not our publish date. Looks good then.
nit: sort `use`s.
I'm not quite sure about the difference to the field *confused*

On Thu, Oct 11, 2012 at 2:02 PM, François Thillen
<notifications@github.com>wrote:

> In src/main/java/org/openengsb/core/api/TransfertMap.java:
>
> > +
> > +import java.io.Serializable;
> > +import java.util.HashMap;
> > +import java.util.LinkedList;
> > +import java.util.List;
> > +import java.util.Map;
> > +
> > +/**
> > + * Replacement for Maps. With normal Maps the WSDL can not be converted to C# files
> > + */
> > +@SuppressWarnings("serial")
> > +public class TransfertMap implements Serializable {
> > +    private MapElement[] elements;
> > +    // If only one field exist, .Net removes the code and replace
> > +    // TransfertMap with MapElement[]
> > +    private int filedforDotNet;
>
> Well then I have to parse the message and then we could also parse the cs
> and replace there Map type with a Dictionary. Then we can use Maps on java
> side
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openengsb/openengsb-api/pull/6/files#r1817626>.
>
>
nginx uses `static` because it's in C, but we have C++ here so an anonymous namespace would be better.  I think it would be clearer if I make the change in a separate follow up pull request though.
The `const_cast` here triggers my "dangerous code" alarm. It looks like `station_` isn't actually used though, so this can probably just be removed.
This operation is not very readable.
Is this enough to check that the system is identified? 
There are exclusion restriction that need to be satisfied, I think.
Need a helper function or method somewhere to check identification of the system.
next time ...
not sure. need to read more about synchronized. pre-existing userLoggedIn was synchronized
added here and there too
TBH I just lifted this from Rack.
This might be a more general issue for later (and connects to a comment I made elsewhere a few minutes ago): it would be useful to have some kind of warning if too many photons are falling outside the range of the input Image.  I understand that it's probably too much for a first iteration of photon-shooting and isn't necessary for this pull request, but shall I open a new issue for this for later?  Or is there some reason why this might be a bad idea?
(to put my money where my mouth is: since Barney is away, I volunteer to remove DoubleGaussian from scripts, tests, etc., if that's the way people want to go)
In fact, as this seems to be combining an inner and outer PSF component much the same of as the use of the `DoubleGaussian` below, this makes a good naming convention to adopt.
Hmmm. Good question.
It's in section "N", under the entry for "name".
It links to the documentation for ```iris.fileformats.rules.ConcreteReferenceTarget.name```.

I believe ```name``` has been an attribute of ConcreteReferenceTarget since it's introduction
so another question is why was it missing in the first place?
What happens if null hostname is passed in? Does this throw NPE, which is caught and logged below to return null?
Perhaps explicitly throw NPE if the user passes in hostname==null, as that sounds like a programming error we should report rather than try to work around.
Why remove the field from AbstractMemento, to have setCustomFields() instead?
Do we need different behaviour is sub-types?

Having the setCustomFields, even if only protected, makes the class look like it is mutable which is misleading. I'd prefer just declaring the field + getter in AbstractMemento (and in its builder).
no longer need explicit apt-get line
Working in dev... haven't deployed yet.
I think this should be declared as `NSDragOperation` then we can avoid this casting.
This is application logic. Put it in SupportedUrl.
한글로 번역하는게 좋은 것 같네요. 다른 번역에서도 테마 이름을 홀로 테마로 바꾸는게 좋겠네요. `Holo.Dark.ActionBar`와 같은 형태가 아니면요.
This will work fine, we are not connecting to anything.
Ok. Thanks for explaining
Because the others weren't broken.

Lee

On Fri, Apr 13, 2012 at 4:34 PM, Tony Spataro <
reply@reply.github.com
> wrote:

> > @@ -88,7 +88,13 @@ def warn(message = nil, &block)
> >        severity, message = filter(WARN, message)
> >        meth = SEVERITY_TO_METHOD[severity]
> >        raise ArgumentError, "Filter emitted unknown severity
> #{severity.inspect}" unless meth
> > -      @actual_logger.__send__(meth, message, &block)
> > +      if @actual_logger.respond_to?(meth)
>
> Hmm .. why the special treatment for FilterLogger#warn but not for other
> methods e.g. #error #info etc?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/rightscale/right_support/pull/34/files#r684587
>
To make possible for Amarok's Query Maker to use all its power, I've created a new database command that wasn't available in tomahawk.
hmmm, does this work?  Usually I've seen multiple selectors separated by commas in a single string, like this:

http://api.jquery.com/multiple-selector/

I tried this in a console in FF, and this was my result:

```
>>> $(".name, .signatures")
[span.name, h4.name, ul.signatures]
>>> $(".name", ".signatures")
[]
```
data-title == 'WiFi'
Missing "your"? `(username) posted an answer to your question “(question title)”`
It doesn't. `$foo['bar']['baz'] = true;` is valid for example
Using a dot in the node name is really weird, and will forbid using XML for the app config (as dots are invalid in tag names). You should use an array node named ``sundown`` with 2 children instead.
I'm not sure I like this approach to updating the objects if there is any other way.  It seems very prone to race conditions etc.

So when we call updateObject that eventually triggers an object->unpack which emits the objectUnpacked and objectUpdated signal.  The problem is that we will then catch that objectUpdated signal and transmit the signal back through the relay and it will continue forever.

The signals a UAVO emits are:
>    void objectUpdated(UAVObject* obj);
    void objectUpdatedAuto(UAVObject* obj);
    void objectUpdatedManual(UAVObject* obj);
    void objectUpdatedPeriodic(UAVObject* obj);
    void objectUnpacked(UAVObject* obj);

so I think if you connect to the objectUpdatedAuto instead it should have the desired affect.  That will means any manual setData gets sent.  Then if you also connect to objectUpdatedManual any calls to object->updated() will also get transmitted.  However receive an object over the link does not cause either of those signals.
make it ccs.neu.edu
They are not the same thing: your if statement and mine handle an empty environment variable differently.  I coded it the way I thought made sense (to treat an empty env variable as an unset env variable) but I am happy to change it.  I'm not sold on the best behaviour either way.

    [cwolfe@cwolfe tmp]$ ruby --version
    ruby 1.9.3p327 (2012-11-10 revision 37606) [x86_64-linux]
    [cwolfe@cwolfe tmp]$ cat /tmp/env-test.rb
    if ENV.has_key?("AEOLUS_CLI_CONF") and !ENV["AEOLUS_CLI_CONF"].empty?
      puts "TEST1 true"
    end
    if ENV.has_key?("AEOLUS_CLI_CONF")
      puts "TEST2 true"
    end
    puts "Goodbye!"

    [cwolfe@cwolfe tmp]$ unset AEOLUS_CLI_CONF
    [cwolfe@cwolfe tmp]$ env | grep AEOLUS_CLI_CONF
    [cwolfe@cwolfe tmp]$ ruby /tmp/env-test.rb
    Goodbye!

    [cwolfe@cwolfe tmp]$ export AEOLUS_CLI_CONF=
    [cwolfe@cwolfe tmp]$ env | grep AEOLUS_CLI_CONF
    AEOLUS_CLI_CONF=
    [cwolfe@cwolfe tmp]$ ruby /tmp/env-test.rb
    TEST2 true
    Goodbye!

    [cwolfe@cwolfe tmp]$ export AEOLUS_CLI_CONF=not_empty
    [cwolfe@cwolfe tmp]$ env | grep AEOLUS_CLI_CONF
    AEOLUS_CLI_CONF=not_empty
    [cwolfe@cwolfe tmp]$ ruby /tmp/env-test.rb
    TEST1 true
    TEST2 true
    Goodbye!
This is analogous to similar methods in other test helpers...

I'll add a comment.
It might also be better to define the regular expression outside the loop.
Наследоваться от TestCase не нужно
No, all asserts need to be triggered only in certain environmentMode's. If such as an assert should run, the client should call it after calling clone().
Pls remove the todo
From my quick research, it seems that yes, the initial user 'postgres' is created, but with NO password. So maybe this line should be changed to one that sets the password "secret" on the extant user postgres? Or are we not doing anything that calls this user with that password?
https://bugzilla.mozilla.org/show_bug.cgi?id=776882
Cvan explained in this bug the current situation of the popular sections elements.
I think we should modify the test so that it checks that 9 applications are displayed on the page, and that the last one is hidden.
I'll do it. I've already started and it's more work than I thought.
`learning_rate` => `loss`
`self.estimators_` is guaranteed to be non-empty since `self._make_estimator()` had already been called.
missing whitespace between "affinity" and "(adjacency)".
Is it intentional because it's necessary, or because that was the maintainer's style?  If it's not necessary, then I would like to point out that it's inconsistent with the entire rest of the file (and the majority of the codebase).  And I do think it's worth touching even just this one instance, because I was already touching the file for other changes.  Whereas, it doesn't make sense to change this in every location that involves files I am not already editing.  I call this "leaving the code in a better state than you found it" or "opportunistic refactoring".  For instance, you could argue that if I'm going to clean up header dependencies, I should only do so if I am willing to do it everywhere, rather than for just one file.  But I don't think that is the bar we should set for refactoring.
Where do these come from ?  Is it from a document or you measured these with measuring tape?

These should all be calculated from the Display instance, there are likely to be more devices in the future, so we can't hardcode these.  If we're to keep some default values in case we can't get the actual values then that needs to be clearly documented.  DPI should be polled too.  If it's not available yet then a TODO is in order along with a JIRA task to track that TODO.
Think i got it thanks
Il faut corriger ce About
If im not logged in...can i see the info about all students?
Please revert everything here
I tend to create un-Owned ptrs, if they are owned by the containing object, not shared and are v. frequently accessed. 
Simply, to avoid the extra object and access.
what would be the benefit of an interface if the methods are not part of the interface ? You still don't know what you can do with the object you have. So it is pointless. We cannot have a meaningful common interface between a builder for DQL and a builder for a mongo query.
Btw, this is also why ``createQueryBuilder`` is not part of the ObjectManager interface, even if all doctrine projects have such a method to get their own builder.
The sub line has been removed.
typo 'default`
_to_instances and _to_values are really not the same things. The former instantiates classes with the values returned by redis, and the later ask redis to return exactly wanted values, without instantiating class, which is really faster.

By the way, i think we should move instances+values+values_list in contrib ! What do you think about it ?
This is there by default too:
https://github.com/mytardis/mytardis/blob/master/tardis/settings_changeme.py#L302
For now, we are not supporting connection strings on servicemanagement. Will come later.
Compass te fournit des mixins pour ça :)
http://compass-style.org/reference/compass/css3/gradient/
Ce serait bien de se faire une petite architecture de dossier CSS. Nottament les mixin utilitaires ailleurs que dans ```application.css```.
Tu peux lire ça http://compass-style.org/help/tutorials/best_practices/ pour avoir quelques pistes sur l'architecture.
@yuhunglin can we write something like this instead:

```javascript
courseIds = _.map(user_enrollment[0], function(key, value){ return value.course_id; });
```

Example:
```javascript
_.map([1, 2, 3], function(num){ return num * 3; });
```
Since there are only two fields, I'd vote for: 

    ['saveAssociation', 'loadAssociation'].forEach ...

as consider checking that both keys exist on mixins

    throw new Error(format('OpenIDMixin requires a %s function', key);
I agree that this sucks. The root of the problem is that the manifestmerger jar is loaded dynamically from the sdk. Hopefully we can soon replace this with the builder library and throw all this code out or the manifestmerger jar will be deployed to the central repo and we can just add it as a dependency. That would also allow us to toss most of the boilerplate out. For now however .. thats what it takes..
why not directly inline this inside cacheFileName?

(QString rawCachePath = cachePath() + QDir::separator() + "raw";)
msf  post(mimikatz) > show options

Module options (post/windows/gather/credentials/mimikatz):

   Name               Current Setting               Required  Description
   ----               ---------------               --------  -----------
   MIMIKATZ_BIN_DIR   C:/metasploit/msf3/data/post  yes       Directory where mimikatz binaries aer stored
   SESSION                                          yes       The session to run this module on.
   SHOW_SYSTEM_USERS  false                         no        Show the system users, which often have un-readable passwords.

msf  post(mimikatz) > set SESSION 1
SESSION => 1
msf  post(mimikatz) > run

[*] Running module against DRAKE
[-] Post failed: Rex::Post::Meterpreter::RequestError stdapi_sys_process_execute: Operation failed: The parameter is incorrect.
[-] Call stack:
[-]   C:/metasploit/msf3/lib/rex/post/meterpreter/extensions/stdapi/sys/process.rb:164:in `execute'
[-]   C:/metasploit/msf3/modules/post/windows/gather/credentials/mimikatz.rb:164:in `exec_in_mem'
[-]   C:/metasploit/msf3/modules/post/windows/gather/credentials/mimikatz.rb:63:in `run_mimikatz'
[-]   C:/metasploit/msf3/modules/post/windows/gather/credentials/mimikatz.rb:54:in `run'
[*] Post module execution completed
I'm writing the commit message for the new version, which does have this kwarg, now - it could probably be removed and replaced with a warning to pre-process (or at least pre-scale) each channel in the next commit if preferred.

As for the depth kwarg, I work with medical image volumes. Often the in-plane dimensions differ significantly from the out-of-plane depth / slice thickness.  Without a depth parameter, in calculating gradients the inherent assumption is that all values are equally spaced.  This is usually true in-plane, but often very much untrue in the 3rd dimension (factors of >= 3x are actually quite common!).  Modifying values through pre-processing would change both in-plane and out-of-plane gradients... I can't think of any way to account for this without a depth parameter in the gradient calcs.  Maybe it's possible, but not clear to me!  If you can think of a way, I'm all ears.
ACMEType: ACMEType_rendered_extension

default_limit: 50
Add `=undefined` for all these parameters since they're optional?
What about tests for the exceptions in `addImage` and `addImages`?  Also consider checking the code coverage; see the [Contributor's Guide](https://github.com/AnalyticalGraphicsInc/cesium/wiki/Contributor's-Guide).
This is the function I was talking about in my previous comment.
Disallowing undefined means I don't have to check it anywhere else before I access the length.  String literals appear to be interned, so there's no extra memory if you have a bunch of blank label.s
I'll remove that from the doc. It should be greater than or equal to zero and less than one. The globe would spin forever at 1.0 and get faster over time if its greater than 1.0. I don't think there are any uses for that. Shouldn't it always slow down.
Same question here about unused settings.
Note to self. We really should consider a childrenCount property to shortcut the copying that would be done here if the counts differ.
fieldcount is also on android, and read-only
@jjinux This is 'old' code by now. The intent was to help with the javascript deployment of the samples. I know @kevmoo and @justinfagnani have both been working on build tools. Long term we might want to employ [buildtool](https://github.com/dart-lang/buildtool) or [bot.dart](https://github.com/kevmoo/bot.dart)
Also, note the [jquery coding standards](http://wiki.jqueryui.com/w/page/12137737/Coding-standards). Specifically the spaces between parens and variable and the tab indentation.
I don't believe that would work because packages_array is an array of hashes, each hash representing a package/repository, we're selecting based on the :name key.

To use intersection it would be 
```ruby
puppet = packages_array & [ {:name => 'puppet', :repo => 'git://a/repo/we/cant/be/sure/of', :rev => "uKn0WNR3visiOn'} ]
````

The difference method ` - ` despite being different, I think, is much nicer then spelling it out in a `#reject` block.
So, just trying to understand... the Symfony2 CMF needs to store its content, so a *something* is needed (and PHPCR-ODM is just one possible provider of that). If so, what is the name for *something*?

Depending on the answer to that, it might be good to explain something like: "Symfony2 CMF needs a *something* to store the content. One example is PHPCR-ODM, and this tutorial explains how to set up PHPCR-ODM ready for use with the CMF".
Я думал вы сошлись на одном. Ок.
Good advice. I managed to get option a) working. Now appender() is defined as:

///ditto
Appender!(E[]) appender(A : E[], E)() {
    return Appender!(E[])();
}
I'm surprised to think that anyone would think that `can not` is good English, and I'd definitely correct it if I ever ran into it in Phobos anywhere. But I don't see much reason to care about "cannot" vs "can't." Pick whichever.
Even without a global `DollarToken`, you can still declare a type here to use rather than calling `_source.opDollar` which risks running afoul of -property, since there's no reason to declare `opDollar` as a property, and you have no idea whether `_source`'s type did or not. Really, if you're ever calling `opDollar` explicitly, you need to rethink what you're doing.
I like these integration tests :)
This should not have the jboss-as- prefix. It should be deltaspike-projectstage to match the folder name.
これは何をテストしてるんでしょうか。
maybe it's a good idea controlling this via a property for backward compatibility?
I don't think so, because the enhancer need other services of the OSGi environment like the edb service or the model registry and I wouldn't put such a code in a model object. Also if we don't make a difference between a normal model and an engineering object, then there is no extra class which is extended? Or do you mean that I should check in the weaver if it is an engineering object and put some of the logic in that way to the model?
You should be using PHPDoc syntax to define parameters (don't do the `is the` portion)

````
* @param $user idnumber of the user.......
* @param $course idnumber of the course....
````
Para verificar que os dois períodos foram removidos!

Em 14 de abril de 2012 18:47, rodrigovilar <
reply@reply.github.com
> escreveu:

> > +
> > +             periodo.numero = 3
> > +             periodo.save()
> > +
> > +             def p3 = Periodo.list()
> > +             assert p3.size() == 1
> > +
> > +             periodoCadastrado = p3[0]
> > +             assertEquals(periodo.numero, periodoCadastrado.numero)
> > +
> > +             periodo.delete()
> > +
> > +             p3 = Periodo.list()
> > +             assert p3.size() == 0
> > +
> > +             p2 = Periodo.list()
>
> Porque esse assert de novo?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/rodrigovilar/AlocadorHorarios/pull/32/files#r685460
>



-- 
[]´s
Sérgio Campos
Graduando Sistemas de Informação - UFPB_LN
пиши нормальну назву плагіна https://github.com/stfalcon-studio/ls-plugin_seo/blob/master/plugin.xml#L4
How about

`dates.sort(function(d1, d2) { return d2 - d1; })`
@Coneko I'm having that problem that version's are not retained by any block if I use the static method to remove code duplication. 
I tried using __block pointers but it's still crashing. How did you fix that?
the ``query_builder`` option is useless as you don't use it and the parent form type (``text``) does not use it either
are you going to implement these?
As we are only adding additional support (not new services), I think 0.3.1 would be more appropriate.
Let's use `.closest( selector )` instead of `.parent().parent()` so that markup changes don't break the code.
Diese Funktion kann ja vom RESPONSIBLE_PROOFREADER aufgerufen werdne.... das heisst er bestimmt die neue Deadline???
I'd call this `import_tags` instead
As this is a third party css, we probably shouldn't reformat it
The implementation as it is now is probably good enough since it's taking into account these variables but we can revisit the area when we are making refinements.
A similar map for remote transaction is also held in the TransactionTable. Similar as in the case of local transactions I think that map can be reused with some additional code changes.  That maps holds <GlobalTransaction, RemoteTransaction>;  this one was added to keep  <GlobalTransaction, RemoteTxInfo>. What can be done is:
- merge RemoteTxInfo into an new class, TotalOrdeRemoteTransaction which extends RemoteTransaction
- change org.infinispan.transaction.xa.TransactionFactory to create instances of TotalOrdeRemoteTransaction when total order is in use.
- then whenever you need to lookup a remote tx, just look it up in the TransactioTable and down cast it to TotalOrdeRemoteTransaction
Wdyt?
Do we rellay want this in this case? Almost all of these are keywords and would end up with a underscore at the end. Will look quite ugly when using these in code.
Will fix during merge
We can perhaps rephrase the above para as below.

For example, You may use the retrieving outbound gateway which receives the message with a user id
on it's input channel to query the database, retrieve the <classname>User</classname> entity and pass it downstream for further processing.
Please indent this line by 4 more spaces and move the operator && to the next line.
Hm. This is JavaScript, you can do `v.m`. :-)
I do not agree with first paragraph, but I find jQuery queues really interesting. Thanks for the suggestion, it will also improve readability and I plan to include them in the next iteration.

On 01. 10. 12, at 23:11, Mitar <notifications@github.com> wrote:

> In tests/test_project/test_app/static/test_app/js/datastream.js:
> 
> > +            'lines': {
> > +                'lineWidth': 1
> > +            }
> > +        }
> > +    };
> > +
> > +    var flot_defaults = {
> > +        'url': datastream_location,
> > +        'datastream': null,
> > +        'from': null,
> > +        'to': null
> > +    };
> > +
> > +    function getMetricData(metric_id, granularity, from, to, callback) {
> > +        if (query_in_progress) {
> > +            setTimeout(function () { getMetricData(metric_id, granularity, from, to, callback); }, 40);
> It's not about errors, it's optimization.
> 
> I am saying that it is error prone. Doing timeouts manually can led you do pile of things executing at the same time, overriding state of each other (no locks in JavaScript), thus things breaking apart randomly. Also that arbitrary 40 there is a sign of this. Because, what if it gets more than 40 ms, then it breaks again to the user and it requests data twice. You need more concrete solutions if you really want to prevent duplicate downloading.
> 
> So yes, try queues. Or jQuery "futures" concept also looks interesting. You could define dataset to be future and others then block until it is available. Something like this. But have not used them much.
> 
> The issue with queues is that they are really queues, so you cannot remove one entry in the middle of the queue, you can clear the whole or pop one out. But you can store somewhere else some flags which tell if data is already available. (But make sure you don't use fx queue. Use your own.)
> 
> I use .data() on DOM objects. So if data is already there, then don't download it second time, for example.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
Personalmente suprimiría `para cada etapa`
its in the standard library, are we supposed to be wholly self contained? Though it might not be in the release version of 3.2.1, I know I have updated my copbl from svn manually for something I just don't recall what.
Missing Doc block
``sonata.cache.phpcr_odm.event_subscriber.default`` is not define inside this bundle ? the ``phpcr_odm.xml`` is missing !?
This should default to 'S' to preserve the previous behaviour.

```
add_field('trxtype', options[:transaction_type] || 'S')
```
D: will fix.
no space between brackets in anonymous functions
why not a simple setter instead of twisting your head around double negation.
``setBuildingHealthRegen(true)``
the same for mobs
I had another joke but let's settle for that one... and I'll change the name
Уже работает в Геометрии.ру
Yes, there is a patientsummary service.  Why didn't you use it?
I don't think they should - it'd be better if they received consistent exceptions so they don't need to worry about at AtomSerialiazationException today but a JsonSerializationException if we change the implementation.
I am not sure, but I think it requires to state units as well. So it should look like:
```javascript
$dialog.css('padding-top', (parseInt($dialog.css('padding-top'))*4) + 'px' );
```
oh okay I can try having it read from the S3 File directly?
Acceptance criteria (https://github.com/blackberry/BB10-WebWorks-Framework/issues/133)
outlines this:
- WebWorks platform additions to the menu (eg. Save Image) should be placed above the menu service returned items.
- Application additions to the menu (ones added by the end developer) should be placed above the WebWorks platform additions.

But if the UI spec says different, I think should follow the UI spec.
I don't believe we need this line for id
Is there anything against just increasing the dialog's zIndex from 100 to 120?
@mdandrea Mock `utils.mixin` function to return the config data you want. This function is called in config.js.
Why not write a file to disk rather than requiring adobe reader?

I would suggest adding another test case for local:/// file access
Part of the tech debt to get the correct webview IDs vs hardcoded values
remove this and move message to convert getDBO's NotFoundException to this message
To be consistent with other DAOs, this interface should not know about DBO objects.  I.e. DBO objects are in lib-jdomodels, and lib-models is independent of that project.
I think the comment about the class here may have been copied over incorrectly or not removed; since the class is set on a different element.