labofsahil/github-event-dataset-2012 huggingface dataset:
I did it... but I have this erro when I run make command.

  DEP  build/types/str.d
  CXX  build/types/str.o
types/str.cc: In static member function ‘static void clever::String::at(const clever::ValueVector*, clever::Value*, clever::Value*)’:
types/str.cc:116: error: invalid conversion from ‘char’ to ‘const char*’
types/str.cc:116: error:   initializing argument 1 of ‘std::basic_string<_CharT, _Traits, _Alloc>::basic_string(const _CharT*, const _Alloc&) [with _CharT = char, _Traits = std::char_traits<char>, _Alloc = std::allocator<char>]’
make: *** [build/types/str.o] Error 1

About this url, it was oppend every time! :) thanks!
Don't commnent stuff out, just remove it instead. :P
:D

The latter idea - using with_file_content. I think that's much simpler, isn't it?
Some docs about returned value? :)
Luckily don't have to worry about different browsers, as all public facing requests go through varnish.  It appears as though varnish uses its own version of the document for `max-age` seconds, then sends a new request with an `If-Modified-Since` header, allowing the server to respond 304 if it wishes. Which is exactly as I'd hope and expect (but always worth trying).
@myleshorton Can jetty's static file server be configured to follow symlinks? If so I can replace the images in this commit with symlinks into install/common (git handles them properly)
did you think that this could be a contention point , especially being static ?
Maybe we could share an instance in the scope of this TM ?
Don't really know, another oversight I guess. Was supposed to be for #1.
ファイルの配置をかなりいじってたりするので，迷惑だったらごめんなさい．
Can you set it that way?
et bibizz découvrit le copier-coller ...
Made the exact same changes lol https://github.com/PhillipTaylor/LibIPvX/commit/f630188a8eedc00f3abedac7d48b88a46f4ac52f
@warpten

Désolé de l'oubli ^^
After speaking with someone at work on this problem, we need to manually fix the relationship between the group and the group profile so that it will be correct in both directions, rather than missing in one, when the controller code is run.

Seems that http://stackoverflow.com/questions/3648699/factory-girl-has-one-relation-with-validates-presence-of has the answer.
Thanks for the info, i was just basing my changes on the changes that lwiddif had listed on his builds

http://loki.spydernet.ca/files/
为啥你没把肉林3的配音加上？
These probably shouldn't be global functions...I think we've managed to eradicate them from our code base (except for the operator<< functions).
The problem with envenom and relentless strike is that relentles is proccing for Effect 1 and Effect 2 of envenom, which are the Envenom buff. I think the buff is proccing twice, but cant confirm.
Indeed there used to be a check first that the compiler was already an MPI compiler.
This should be reintroduce at the beginning of DetectMPI.cmake.
It was done by trying to compiler a simple MPI program with the CMAKE_C_COMPILER and CMAKE_CXX_COMPILER
shouldn't that be 

    ->field('_id')

?
ok.


--
Juanito



2012/2/4 fengdidi <
reply@reply.github.com
>

> ҶˣOKҵǸ޸ٰ޸ĵĸͨ˳ˡ
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/fengdidi/acl-chinese/commit/25710b02f6cc957f92915629d0d3a6aa2f097699#commitcomment-932936
>
Ah, so you added it :)
After this, glob should probably be renamed, since it can be a glob or an re, but it looks good to me. @wycats?
А собственно при чём тут программирование? Если есть HTML отформатированный по правилам, нафига лезть в него со скриптом? Где сказано что надо обязательно всё на JS формировать?
to be consistent, should be `filter-img!` as it is impure
makes sense. my original class was using interfaces, I like the idea of switching to annotations, but let's make it uniform, using @PreDestroy
Arthur

This returns an integer that relates directly to the constants, but how do we more-clearly document this?
Yeah - I'm thinking of  adding a -(BOOL)testForNamedFile: to do the work of DRM sniffing.
As far as I understand the problem is more complex than we have imagined. There are a lot of compatibility problem to consider with specific terminals. I'm studying the "tpope way" in vim-rails and it's actually a difficult problem to solve. I've created a new branch in order to solve the issue in a focused way. Of course, if you want to help it would be great. I have no time today but tomorrow I'll push a first version of the code I intend to use. I'd like to create a function that will be responsible to execute the job in background. In this way, I'll be able to support even windows (LOL) in the future.

P.S. we have a very simliar setup. I'm a zsh user too but I use gnome-terminal. I tried with bash and the fix I made it's working fine. So, I think the *real* problem is with terminals (at least on *nix machines).
it should be a reference loot, so you should change the above to reference_loot_template
Note to self: Read all the things, then comment.
@soloweb 

>, да и зачем нам лишнее борохло которуе уже устарело.

Это "борохло" используем мы и, возможно, кто-то еще. BC ломается. 

>Что-то про такое я не слышал, что он там что то сам режет и ничего об этом не говорит, странно.

Попробуй сделать get ключа из 1000 символов в консоли и через pecl. А tcpdumpом посмотри что фактически получает memcached сервер. Результат удивителен. )
+ в pecl memcache есть дополнительное кеширование afaik, т.е. не каждый get вообще будет отправлен на сервер.
@3kids stop being such a dick and actually contribute something or gtfo
Are you sure or are you assuming they are in order?
only cleanup
wrong branch:)
sorry, seems to be lost. cant find it either.
I'll send another pull request later fixing this, if you want :D
Wehn I started off coding the library, I've stored the same fields as the original media library does; this was one of the fields. It seems this has been changed to be computed directly when a stream is being requested.
django 1.3 syntax
Hey Peter - I still am not able to build -- is the new SNAPSHOT live yet?
I thought players couldn't do html tricks with their names?
Wireframe for the Refresh App by Daniel using http://pencil.evolus.vn/
OK, I see.
Thanks for this tip.

As I've told before — I'm not a magician... Just learning. :)

Sorry for bothering and thanks again.
Rebases are problematic because they should only be done manually on non-published commits. Merges are preferable.
Example: The device's resolution is initially 400px, but the lowest Breakpoint has been set to 640.
Cool.  I was looking at the source earlier today and noticed that you
are trying to create an HTTP stream and attach it to a VideoViewer
instance.  It's too bad the HDHomerun doesn't produce a standard RTSP
stream or run a small HTTP server that would serve the stream you are
trying to create.  Interested in helping with this project when I can,
but I don't have much time with a full time job and a small child.

-Eric

On Fri, Jan 6, 2012 at 8:16 PM, drhill
<reply@reply.github.com>
wrote:
> Haven't worked on it in awhile.  I started to see if I could setup a thread
> to receive the UDP stream from the HDHomeRun Prime and then re-transmit the
> packets as an http stream.  I was getting data from the prime, but not
> successful doing anything with it.  I'm going to try to pick up work on it
> again soon.
>
> On Fri, Jan 6, 2012 at 3:30 PM, Eric Work <
> reply@reply.github.com
>> wrote:
>
>> How far did you get on this before you stopped working on it?
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/drhill/libhdhomerun_java/commit/b061e9c5280fedf6ad828b26193e93f5c004cf3d#commitcomment-844850
>>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/drhill/libhdhomerun_java/commit/b061e9c5280fedf6ad828b26193e93f5c004cf3d#commitcomment-845640
looks good. I think it should
Mark, the new approach is here: https://github.com/olegz/spring-integration/commit/0492929b9e8152aabe49ad0915c486eebf35a5c5
IMHO its much better since everything is resolved naturally at the appropriate time. Commit log message has all the details
This is unrelated to gcc.
Hello,
thank you very much for the fix. I tried to install the ThemeEngine using the knplabs symfony2-with-composer http://knplabs.com/blog/symfony2-with-composer but it doesn't retrieve the required bundles alphalemon/al-pagetree-bundle and alphalemon/al-valum-uploader-bundle: could you give another look to the composer.json?

Thanks again
nice save... @sekiskylink watch that.
static is bad!

Instead, see cf_data_add(), and cf_data_find().

this function should call cf_data_find() on the input cs.  Use a key like "tls".  If it finds a pointer, return it.

Otherwise, parse the config section, and then call cf_data_add() to cache it in the CONF_SECTION structure.

This is probably less code, has no static variables, and is easier to understand.
Forgot to .gitignore it on first commit. :S
Changed it :)

Bedankt!
Warum das? Mach doch einfach ein _requires hin und gut ist...
Feel free to improve the code… I won't do it at the moment.

If you do I'm happy to click the merge button.
This logic should not wrap the function like this. This will cause erratic results in the caching and querying of blocks in Drupal core.

Why are we not using the Block visibility settings in the block config page?
nope ) will talk to you about that.
@ericgj Not to continue this discussion on GitHub, but I would suggest looking at the Commands module in the same light that you'd look at Enumerable, Comparable, etc. from the standard library. Both of those modules expect you to implement a particular method, and it's up to you if you wish to do something stateful in those methods or strictly behavioral. 

As a side note, one violation of this general rule is the standard library's Observable module. Unfortunately this module maintains state in an @observer_peers instance variable which could clash if you happen to have one defined in your including class.
By default a new sketch has the width/height of the canvas element.
Still it can be set from within the sketch. If it is not set within the
sketch, width&height contain the correct width/height of the canvas
Previous style preferred.
What if we made a 1.1?
I think we're missing a `../`

    'MediaExposer'  => __DIR__.'/../vendor/media-exposer/src'
Probably don't need to copy: KeyBundles are effectively immutable.
Reformat to be readable, please!
Nice.  You might try this in more concise fashion with

    eclipse.project.natures += 'org.springframework.ide.eclipse.core.springnature'
splitEvery (length testPairs div clients) testPairs
Please enable PDO and pdo_mysql extensions. Also this armory needs PHP to have CURL extension working. Peace :)
arguments**
Did you test this ? working ?
What is this mach-fsm meant for ?
We only wanted to push the code into msm7x30-common and I had already made changes, I uploaded as a single commit, again as Kali requested to view what I was working on. No credits within the source were removed. I've not intentionally mislead anyone and again, this wasn't for general consumption but sticking it on gerrit allows Kali the access he requested and still allows me to push updates as I fix things (ex FFC switching and crashing).
I'll admit that this is lame ...
Removing spaces, use the formatting tools led
Shouldn't this be:
`private final Logger logger = LoggerFactory.getLogger(getClass());`

So that child classes get the correct logging instance?
For this one, the protocol doc actually refers to a `url` parameter! I am assuming the doc needs to be updated to point to service now.
Man, this is becoming more burden :-) 

I don't have any more coop dev hours to burn on these nit picking :-)

Sent from my iPhone

On Jul 17, 2012, at 18:04, Misagh Moayyed<reply@reply.github.com> wrote:

>> +          p:centralAuthenticationService-ref="centralAuthenticationService"
>> +          p:warnCookieGenerator-ref="warnCookieGenerator"/>
>> +
>> +    <bean id="generateServiceTicketAction" class="org.jasig.cas.web.flow.GenerateServiceTicketAction"
>> +          p:centralAuthenticationService-ref="centralAuthenticationService"/>
>> +
>> +    <bean id="sendTicketGrantingTicketAction" class="org.jasig.cas.web.flow.SendTicketGrantingTicketAction"
>> +          p:centralAuthenticationService-ref="centralAuthenticationService"
>> +          p:ticketGrantingTicketCookieGenerator-ref="ticketGrantingTicketCookieGenerator"/>
>> +
>> +    <bean id="gatewayServicesManagementCheck" class="org.jasig.cas.web.flow.GatewayServicesManagementCheck">
>> +        <constructor-arg index="0" ref="servicesManager"/>
>> +    </bean>
>> +
>> +    <bean id="serviceAuthorizationCheck" class="org.jasig.cas.web.flow.ServiceAuthorizationCheck">
>> +        <constructor-arg index="0" ref="servicesManager"/>
> 
> Seems like this is the only new changes for this file. The rest seems to be IDE's formatting noise. Could you clean out just the needed change? 
> If you're using Eclipse like me, I realized that I needed to turn off its 'Save Actions' feature or restrict it to only formatted lines. 
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Jasig/cas/pull/88/files#r1183632
2 independent icon checks so that in case you are hovering mouse over one, or block by option menu (right clicking)
Well, at least no one can say it was a case of C&P :smile:
No comment describing why?
Pattern matching ftw
no, `false` means “not in response to failure”; in this case it means “in response to the `suspend` five lines up”
no, ClusterMessage are the ones that are sent over the wire, and needs to be protobuffed
why not `new InputStreamReader(new FileInputStream(file), "UTF-8")` ?
Well, it can be, but not always
j.u.c.TimeoutException
We usually put these between the package declaration and the code imports, to easily see which extra features are enabled quickly
Yes, but I don’t see any compat issues: since it is the implementation, it should not be overridable, instead users should override `createPipeline`. Then they may call `defaultStack`, so we are bound to its signature, but that’s the end of it (correct me if I’m wrong).
since I feel a lack of comments from my side on this PR is not acceptable, I have no choice but to say :+1:
It has a certain odor
This use of Account is strictly class methods and it is trivial to rename or load it into a different namespace if necessary.  It is also, and will remain the codename for the Set::Right::Account methods and rules, which is why I think the name really makes sense.  There is no reason for it to be in app/model, but that's where it started.

The User model still exists for the low level stuff of our particular account implementation.  As discussed, this can be any class, and doesn't even have to be in our app.  The intent is to proxy the low level functions through Account, and interface to them via Rack/Warden.

As for the controller, that is a different matter, but if we still need a controller besides card_controller, the name could be account or session.  This doesn't lock us into anything there.

I don't think there is a ticket, so we should create one.  You want me to do it?  What should it be called?
yes... any suggestion to clean it up?
On Mon, Apr 02, 2012 at 05:57:43PM -0700, Andreia Gaita wrote:
> > @@ -5749,6 +5756,11 @@ internal override bool SetTopmost(IntPtr handle, bool enabled)
> >  
> >  			Hwnd hwnd = Hwnd.ObjectFromHandle(handle);
> >  
> > +			if (hwnd.topmost == enabled)
> > +				return true;
> > +
> > +			hwnd.topmost = enabled;
> > +
> 
> Any particular reason to prevent SetTopmost from being called multiple times?

Not really, I can change it if you like.

Thanks,
-- 
Peter
Again does this apply to both groups and orgs or just to orgs? I thought the difference was that groups were 'open' things that anyone could edit, so they don't need to have members and roles.
Let's take this out for now. This falls under the same jurisdiction as Pradeep's needs to migrate RPM units. Randy is working on a solution for this, but for now I don't want any references to a specific plugin in the platform.
Can probably just remove these if they aren't going to have values. Anyone using an itinerary will know what these are anyway.
this important change didn't make it over
Move to com.wajam.nrv.utils?
Not if it was part of the class like the other emulated functions..
Please put the actual code on a new line (no { } needed if it's only a single line)
This only evaluates to true if `BARCODE_STDOUT` is the only enabled option.
w is not a great variable name.
and add a comment.
in this case i used javax.ejb.TransactionAttributeType class. if you think it's more correct, i can use a wrapper class.
Useless as it is not a Component.
Should use loop { react{}} or every task will block on thread of the scala Actor thread pool.
And the after string, if any.
Will check out later...

On Saturday, May 19, 2012, Anil Madhavapeddy wrote:

> > +      | Write_barrier -> 2
> > +      | Flush         -> 3
> > +      | Op_reserved_1 -> 4 (* SLES device-specific packet *)
> > +      | Trim          -> 5
> > +      | Unknown n     -> n
> > +
> > +    let of_int = function
> > +      | 0             -> Read
> > +      | 1             -> Write
> > +      | 2             -> Write_barrier
> > +      | 3             -> Flush
> > +      | 4             -> Op_reserved_1 (* SLES device-specific packet *)
> > +      | 5             -> Trim
> > +      | n             -> Unknown n
> > +  end
> > +
>
> You can use 'cenum' for all this btw...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/avsm/mirage/pull/97/files#r849875
>


-- 
Dave Scott
This isn't really a comment for this script, but we probably should fix cyclus to look in the same directory as the input file for catalog files.  Or look in some designated catalog location.
Ok, at this poin should be used `Buffer::New(const char*, size_t)`
@bnoordhuis can I merge with head?
I took a closer look at the documentation of LinkageError and UnsatisfiedLinkError, and it appears to me that their documentation is bad.  To wit, according to the documentation of LinkageError, it doesn't sound like UnsatisfiedLinkError should be a subclass of LinkageError, and the way UnsatisfiedLinkError is used by System.loadLibrary doesn't quite seem to agree with the class-level description of UnsatisfiedLinkError.

Given this imperfect state of Java, I still think that UnsatisfiedLinkError is a better choice here because that's how Java's System class signals that it wasn't able to load a native library, and ultimately the possible failures in our case are versions of that.
Done, thanks a lot!
Yeah, I was thinking of adding a fallback as per your comment in d3.geo.path, but then I wasn't sure whether to enable clipping or not.  It seems reasonable not to have clipping or rotation for d3.geo.albersUsa, at least for the moment, since you wouldn't use it for anything other than the US.
Let us do it and mark createWorker as @deprecated.
Great tests, these really leave this corner of numpy better than you found it...

To be *totally* comprehensive, they should also check that the warning is in fact being issued by these last calls to concatenate -- see numpy/core/tests/test_multiarray.py : test_diagonal_deprecation for an example of how to do this
all time tracking is currently done via time.time() - not the right way to do it (datetime?) but the change should affect more than just this line
Again, this `zval_ptr_dtor` isn't right here (and also the one in line 360 and 377). (And also again the `5` in the above symtable find call).
запятые по стайлгайду перенести
I know this is trivial, but I think `namespace_path` is actually more accurate. The 'namespace' in `namespaced_class_path` and `namespaced_file_path` is more of a prefix to describe the namedspaced nature of the paths. But, in the case of `namespace_path` - the path is not namespaced per se - it is the namespace itself. What do you think? :) @josevalim @pixeltrix
How about this https://gist.github.com/3027391
No need to mention this, I think
We don't need to use merge here. This should be fine

```ruby
options[:primary_key] = primary_key(from) if primary_key(from).to_s != 'id'
```
Why not just `@header['X-Frame-Options'] = self.class.default_x_frame_options`
The length of this line makes me sad :(

plus it makes github draw a scroll bar.
Can we have a more descriptive name for this, maybe mustInlineCalls?
A little Thelonious Monk?
In scala-reflect we ignore the fact that a field or a method is private. We do `setAccessible(true)` transparently anyways. I think we should pursue the same approach here. What do you think?
if `fnorm == 0.0`, you get a `ZeroDivisionError`
``a`` and ``b`` are both a single array, not a sequence of arrays. This type descriptor should be ``array_like``.
I think you may have made a mistake here.
Why does dash need this decorator?
If the mess that is `Function` makes that too hard, could you at least use `.atoms(Function)` to ensure that the names really are new to the expression?
> Well one thing I don't want to do is to force people either into using a predefined set of heading names, none of which really match what you want, or no headings at all.  That definitely doesn't lead to a good doc reading experience.
>

Perhaps we are talking about two different things? I thought (through
the education that Stefan, as I recall, was adding during the code-in
this year) that there are some existing labels (Examples, Notes,
etc...) that Sphinx recognizes. So that handles the "no headings at
all" case: there are existing headings. The whitelist would contain
the few additional ones that *more than one person* (as confirmed in
the review process) thinks is useful. And that would handle the case
of not having ones "that really match what you want." What's nice
about standardization (even in the case of using Examples when there
is only one example) is that it gives a standard way of looking for
things (e.g. with grep) or referring to things (e.g. "look at the
Examples section to see one or more examples pertaining to the
function" as opposed to "Examples might be listed under Example,
Examples, An Example, Gotchas, Advanced Examples, etc...") What we are
talking about is headings which are the "intellectual markup" codes
that one can use...with flexibility in that some additional ones can
be added to a whitelist.
"Symbol's" feels both verbose and awkward.
What happens if I have a symbol like "Conductance"?
now I see the extra condition...but you already have the first condition true (by merit of the first test) so the additional clause of the `or` at that point is superfluous. That `or` needs to be appended to the if-block a few lines up.
`"fails for %d*pi/%d" % (i, n)`
these additions can be removed since we are already inside that condition. You've done:

```
if foo:
    if a: return
    
    process...

==>

if foo:
    if a: return

if foo:
    process...
```
sp: Imaginary
corrected to use `default_sort_key` when quick is not True
Accessing to booleans, doubles/floats and integers by indices is not a problem too:
```php
error_reporting(-1);

$v1=true;
CVarDumper::dump($v1[0]);echo "\n";

$v2=123;
CVarDumper::dump($v2[0]);echo "\n";

$v3=123.123;
CVarDumper::dump($v3[0]);echo "\n";

$v4='test';
CVarDumper::dump($v4[0]);echo "\n";
```

Result:
```
null
null
null
't'
=
```
Geometric field isn't actually used in this routine.
seems redundant.
spaces around + signs
Yeah, I prefer to put .gitignores in the respective subdirectories as needed.
just about redability: ``` list($serviceId, $method) = $callback; ```
Why ````texty````?  Can we replace with something that is easier to read?  Also, please move to the CSS file.
brief is the goal here: there'll be another document for running MongoDB in a more production-like context.
you don't need the customer
I did not realize driver interface was under driver. Perhaps this is fine.
It works fine in Windows.
The variable name no longer quite reflects its purpose. It should be maybe tsearch_usable. tsearch_present was my first thought but technically it is present in the old versions, but we won't use it.
If either way works, don't change the superglobal, copy it into a local array and modify the local array.
No more global. Use dependency injection via the constructor (if object is required) or setter (if object is optional).
Might want to get rid of the ugly `++` here. This involves adding another variable. Same for method below.
I disagree. "your PHP configuration" is consistent with already existing language string and still works for shared hosting where the configuration can not be changed.
It's really rather easy: Enable post count in forum X, post in forum X, user_posts is now 1, disable post count in forum X, delete post. User now has no posts but user_posts is still 1.
And you are actually not returning by reference here, again according to my understanding.
Check the function you need.
Well, I think a switch statement don't need necessarily a "default", especially when the default value was previously set (in L470). You already implicitly assigned undefined.
I wonder if "wrench" and "yuidocjs" should be in "devDependencies" instead?  (That is, neither of those are needed in production.)

Hmmm... perhaps there should be a separate "mojito-dev" (or "mojito-tools") package that contains development-time things (and let "mojito" just contain the runtime parts).
In this case, filename is the file that jslint was run on. The logic for instantiation differs slightly: https://github.com/yahoo/mojito/pull/199/files#L0R364

This variable ensures that I write the filename only if I write an error to the screen after instantiation.
1. Нет, проблема совсем в другом(хотя запросов будет действительно много), и она не имеет отношения к ноде. То же самое верно и для php. Рассмотрим твой запрос. Ты хранишь количество показов в переменной и записыаешь её в базу, соответственно, когда будет несколько нод, они просто будут перезаписывать значение друг друга. Считывать текущее значение, а потом записывать его в базу, прибавляя единицу тоже не вариант, даже если делать это в одной транзакции - придется выставлять уровень изоляции транзакций в такой, что их можно будет писать в базу только по одной, что убьет всю производительность.

Когда пишутся интервалы, они совершенно независимы. Мы можем добавить ещё 10 нод, потом убрать половину и все равно система будет стабильно и правильно работать.

2. Да, по каждому промежутку. За день инфу писать плохо по нескольким причинам: нода может упасть и потеряется инфа за весь день; чем реже мы пишем данные в базу - тем более неточно работает ограничение на количество показов; обновлять статистику раз в сутки - это ужасно медленно.  400 записей в день на баннер - это очень мало, такой нагрузкой можно пренебречь.

Правильно было бы, чтобы крон раз в несколько дней/недель делал перерасчет статистики. То есть сливал пятиминутные интервалы в часовые, например. И далее часовые в дневные по мере устаревания информации. мало кому понадобится точная статистика годичной давности. Ну и в таблицу для статистики нужно будет добавить ещё одну колонку enum для отображения периода времени, который в этой колонке содержится.

Тут кстати очень интересная пробелема возникнет из-за удаления записей статистики - придется делать optimize, чего очень не хочется. Надо будет подумать как этого избежать.

3. Очень не хочется нагружать ноду логикой. Да и получается что кадый раз всем нодам придется перелопачивать данные по статистики (которые могут быть достаточно большими).
(need I say more?)
Also:
```
$ opam list *foo*
  'opam list *foo*' failed
Fatal error: exception Re_perl.Parse_error
```

I guess in this case the error message should be more user-friendly :-)
I wonder now if it should not be np.asanyarray    to allow ndarray-subclasses to not waste memory/time
@johnstoecker https://raxaemr.atlassian.net/browse/RAXAJSS-628
\+ private Drug drug, private Provider provider, private Location location, private DrugPurchaseOrder drugPurchaseOrder
In case it's not obvious from the tone of my last comment, and my comments on #86, I've very happy to remove this "hack". I might go for something like:
    # For each of the first four time-steps, pull out the bottom two levels.
    cubes = [cube_4d[i, :2, :, :] for i in range(4)]
代码中是xBase。这里是yBase？
Is this sufficient @josharian ?
Testing vs. Tested?
Can you change this and the other exceptions to Guzzle\Common\Exception\RuntimeException?
=> `left: 1 }`
I would make some ad hoc adjustments here to resolve this here, although I am wondering whether we'd like to place some naming restrictions/conventions to avoid these mistakes in the future.
Sorry about this. Still new to Chrome :)
I might argue yes considering how constraining it is that you can't override package templates.

But the smart package provides a route around making that decision be universally damning.

I've re-imagined a lot of my libs in the last week including this one... in a nutshell I'm calling the smart package template-hooks. The idea being this (and other libs that I'm calling *-hooks) are violent monkey patches that solve an immediate problems for myself. The *-hooks prefix is also meant to imply I don't nesecarily think that Meteor should change but it might be nice if they exposed hooks that would allow me to throw away the smartpackage and get the same behaviour through configuration. Make sense? Unfort. it's gonna be a bit before I can push all these changes out.

-Mike
Oh sorry, I missed that.
`if locale in product_details.languages:` will avoid KeyError. It is possible that the second key (`display_locale`) would raise an error, but this would be a problem in the data. it probably should raise an exception, because it means there is something wrong with the product details library.
I tested it -- val is a list. I think this is bad though, so I will change it to a string. I think it had to do with the way meta content type is normally stored in headers (it's usually wrapped in a list for some reason)
There are only 3 formulas that use openjpeg: oiio, mupdf, and gpac.  All three use `#include <openjpeg.h>` with no version.  I assumed that they knew how to craft the include, and it was my responsibility to locate it.  At first I thought we were installing openjpeg incorrectly, but I can't justify that from looking at our formula.  It just calls `make install`.
Perhaps we need to rewrite these imports in fewer lines?
```python
from vumi.application.sandbox import (
    Sandbox, SandboxCommand, SandboxError, RedisResource, OutboundResource,
    JsSandboxResource, LoggingResource, HttpClientResource, JsSandbox)
```
One of those is the htmldiff output, not markdown and cannot really be applied in the template
Making sure ```__all__``` is defined was sort of the intention.  It prevents things leaking from import * (like g, c and NotFound) by making sure each module has an ```__all__```.  My intention is to only add ```@export``` where the ```__all__``` is already defined, so this is more of a backup.  I can modify this handler though to remove the backup, the end result though will still be the same since part of my changes will be to add ```__all__``` to each modified file.
nit: on same line
Drop the sentence about `forces resolution ... ` . . . that's an implementation detail that we don't want to promise.
My original idea was also to generate single and archive templates for the custom post type too. That's why it's so theme related. But the direction is going more towards general post type scaffolding. 

Let's remove the hooks and maybe I'll work this out in another command that is more theme development specific or a plugin command of some kind.
不需要吧，make之后就已经有可执行文件了，具体在哪个目录下我不太确定，但我基本上肯定是存在的。make install的过程基本上就是把编好的文件们拷贝到相应的系统目录下。
Can you add a comment about why this is necessary?
Scratch that. `getPage` checks for you, so it's unnecessary here.
ah thanks, changed that and also made 2 other changes:

a) moved the if ..... return statements more to the front
b) added an if (player !=null) to the kickbanevent for ban ip

I guess I will push again tomorrow
Been writing a lot of iPhone code lately... does it show? :P
"Inline editor" leads me to think of "Quick Edit" type of inline editor that is located in the editor area.
@jrowny pointed out that if the welcome project is missing we'll end up in an infinite error message loop.
I think you have understood the code here. This block of code has nothing to do with testPos. On line 377 we did scan backwards with _movePrevToken(ctx) and if it fails we return an empty tagInfo. If not, then we're checking what we got when scanning to the left of the current pos. For this pull request, we're adding ctx.token.className === "comment" to see whether we have a comment before the current cursor. The rest of the logic is the same as before and is specifically checking whether we have a begin tag or an empty tag or an end tag right before the current cursor pos.
Good call - fixed.
It does animate cleanly. I'll let @GarthDB determine if it is intentional.
Do PHP and LESS have a similar problem?

Maybe we should just do a quick run-through of all the "officially" supported filetypes in EditorUtils and make sure the output is sane for all those cases?
:+1:  Thanks for writing tests along with your code, very much appreciated
does this need to be public ? is it needed at all ?
Please check if your po editor returns those removed fields back. And if it keeps the character case of the header the same.
use more desciption in this var `options._tickerInterval` makes sense.
Paul Irish did this sort of thing really elegantly with rAF here: http://paulirish.com/2011/requestanimationframe-for-smart-animating/

You *might* care about making it look like that, but not necessary.
Ok I was wrong.
Why not make this an option you can pass in as a parameter to `register`?
I thought you would have preferred just doing string replacements. But, if it's DOM you want, this is good.
Grammar is odd here.
Good old data.data
I noticed you aren't doing anything with this right now, shall we file a follow-up ticket on this?
In stylus, the comma normally designates the start of a new argument to the box-shadow macro. In this case, I want the comma to actually appear in the `box-shadow: ...` statement. Thus, the `\` is used to escape the comma.
Yes, so in conclusion, we just use the http://localhost/CoinCod/  instead?
Might do ```salt-master``` here?
NAzwa bazy frontendowej też tak jak mówiliśmy powinna być w settings.py ustalana
This is okay for now, but we may want to consider using a templating engine like mustache for this kind of stuff. Concatenating HTML strings is kinda janky

https://github.com/spullara/mustache.java/
an RRset is the set of RRs that have the same name and type - for example, 'all MX records for amsterdam.example.com'. Section 3.2.3 requires that we treat the prerequisite section as a collection of RRsets, and make sure that every RRset appears in our zone in exactly the same way.

Example:
our zone has:
amsterdam.example.com. IN MX 10 mail.example.com.
amsterdam.example.com. IN MX 20 mail2.example.com.

prereq in update packet has:
amsterdam.example.com. IN MX 10 mail.example.com.

We check that every RR in the packet is also in our zone - and we find that we do have the prio 10 MX. However, the zone also has the prio 20 MX which is not in prereq. In this case we are required to signal NXRRSET.
Probably worth extracting it as a parameter: if we wanted to reuse the foreman module to be install it on a different machine.
it should skip the test, not throwing an exception
there are some unused use statements
Don't know why I didn't underscore that, good call.
^ remove
Note that a page can only have one debugger instance; if the web inspector is enabled, this will get replaced. But that seems okay, as if you enable the web inspector you're presumably using that for debugging instead of terminal output.
scipy is not a required dependency for astropy, so it should be imported only in the functions/methods that require it.
I think the main reason for this is that the efunc is a lot simpler to write if flat is guaranteed, right?  in that case this still may be worth keeping, *but*, there should be a note in the doc that makes it clear that this is the main reason why it exists.
Well, I was thinking they wouldn't want to do this because of the confusing state of "iterable" between the two versions that it would create, but I guess you're both right that this should be left to numpy people to decide :)
Without catch block, won't the exception just get thrown further?
That's a nice suggestion, perhaps delete the old one if it exists and create a new Bitcoin-Qt.lnk.
Having a destructive get() seems like a bad idea.  Why not just call set() afterwards to clear the block.  I'd like to avoid duplicated functionality.
localhost would be better, these tests should not need internet access to run.
coding standard issue: booleans should be written lowercased (same everywhere in your code)
group: looks like it might want to be set to @subgroup here
Niet beter via updateFeatured($item)?
Any reason why you don't use a static method?
Almost there .. but this preventDefault is still disabling the scrolling of the page itself. I think maybe you are not experiencing this b/c you are viewing this on a regular browser. You should reduce the height of your browser so that the main content are scrolls, or better yet, use the iOS simulator. Then you will see that the page does not scroll b/c you cannot swipe up and down.

I think if you move the preventDefault call inside of the two IF statements, this will fix the problem. I'll test it out and verify and them merge the pull request. thanks!
File was created in 2011.
I'll go with `run` (but private) for consistency.
when writing tests, it is useful to have some assertions!!
`object_count` I think
We could also come up with a matcher that doesn't have this problem. Perhaps `/:channel/:from.:to/transcript.?:ext?`. It doesn't really benefit the users to do this though, which makes me think we should still focus on one of the options above.
Yeah - that was part of the cleanup which still needs to be done.
A docstring here would be nice explaining the redirect logic.
should be indented to same level as <thead>
slug+locale is guaranteed to be unique so you can use `document.slug` + `document.locale` for this instead.
definitely a confusing url!
Look at the desktop template and add a list for the `{% if fallback_documents %}` below. Style the "In English" however, might not even need the style tweaked.
fixed the docstring to be more clear.  the "both" was cases from both prefixes.
``if not params[param]:`` is enough here, ``not None`` will return ``True``.
I just hope that this doesn't slow down frustum by more then 5 percent.
this creates a global variable. It needs to be either static or inside an anonymous namespace
Domestic and International packages have different properties, so it didn't make sense anymore to have a single Package class. So Package is a base class, with InternationalPackage and DomesticPackage inheriting from it.
I changed the message to refer the specific package class.
FWIW, I don't think you need the `\` here since you have the comma.
@Tobion - as I see, the assertion is testing a string compariso, so yes it does matter in this case of course, my bad :)
To be honest I don't really think it's a problem as I've turned the gradient 135 degrees to right so that it looks a bit pretty.
As for gradient prefixes, I've took it from .sf-toolbarreset class so I'm not sure what do to here.
No need to extend ContainerAwareCommand when you dont use the container.
should we typehint then ?
need scope public
@Krastanov If you don't want to store your password, then don't do it. You could type password every time when bot started.
I pushed to both my master and Status branches. Where else do I need to push this to?
it's not strong, it's just less code, which is easier to read/maitain. And less code is generally considered better.

and I think that there is no reason to use `setValue:forKey:` instead of `initWithObjectsAndKeys:` in this case

but again, it's matter of taste, so do as you wish
集鵠大叔。。。你看空格和引号的能力真是一绝啊。。。感谢。
explain this a bit more -- why we need it
You can't have newlines within wiki links. Must be: [[Basic Cluster Setup]]
I like your change, returning only the sessionId simplifies things.  But I think this will break any code using the existing api, because right now it returns the whole object.

I'm a little out of the loop with opentok, you would probably know better than I.  Should we just return the sessionId and change the api?
So initially `isRouting` is false, and on subsequent state changes it's true? Perhaps you could clarify that.
S/brek/break
I think we can simplify a little by just making the definition be `def get_logs(app, options = {})` and removing the following line.
Tämä on nyt ihan sama, irkissä laittama juttu oli vain esimerkki. Nyt viewin funktion nimi on eri, kuin urlissa oleva.
I think this should just be an optional dependency, otherwise normal servers will not boot properly if the ejb3 subsystem is not installed
Only one `var` statement per function.
`Finally`?
Yes it is. This is what allows you to change the momentum over time, which is very important for getting good results. But it would have to be updated to point to the new momentum coefficient.
Thanks for the tip~
I think this app will only display system wallpapers, so I doubt there will ever be any with different aspect ratios.
FTR, `shared/style/edit_mode.css` has been merged.
Right again :) doing it.

Good bless code reviews
Ok, just added:
I think it can work but this leaves the door open for additional code to add there
html_structure -> htmlStructure
I am not 100% sure but the upgrade code in this function assumes load is always going to be used prior to interacting with the UI. That may or may not be true and we should likely now move the upgrade code to open... I recall I initially wanted to place it there but had some issues... 

I think we definitely need to move it now.
nit, this extra line is not really needed
Would it make sense to put this inside Browser.init() alongside Places.init()?
`detachListeners` or `unregisterEvents`, lower `R`.
where does 'compact' come from?
Naming should be `Php` - it's ugly but more consistent.
yeah. these time periods are tiny in practice -- if you don't enter the serialized block directly after adding, it's because another thread is already in there, so that thread will loop back and truncate the queue for you -- but it does mean there can be a few more items in memory until then.
I am still so confused on this... Do you get the error even when your seclude the test via TESTS=./test/test.whatever?
Change this if statement to `return pid or BAD_PID`.
Remove this method since you never use it and add an issue on GitHub, since this method and the above TODO will surely be forgotten otherwise.
The CSS has been generated from the Sass mixin here: https://github.com/alphagov/prototyping/blob/master/_includes/scss/design-patterns/_forms.scss
...which is currently only available from the Prototyping repo. Would be good to discuss how best to utilise the mixin - this contact form is the first product to make use of it.
Will this ever be called externally?  if not, maybe just move it all under ``if __name__ == '__main__':``?
I ommited the term sourcestamp but as sourcestamp is widely used in the documentation it is probably better.
updat_ing_ your dependencies?
all these imports needed?
What is your reasoning behind, adding class 'hidden' and then showing it here. The div is empty by default?
done and securing code for old libc.
Reset to a -1 like value to not close it again by accident?
No, I expect classes to be marked as Extension :

    @Extension
    public class Foo extends FileCallableWrapperFactory { ... }

so indeed AbstractDescribableImpl is not required.

how to use ServiceLoader with Jenkins ?
Constructor should not be public.
Thank you, another line less to show up in the diff to the current platform ;)
Yes.

-Jeremy


From: Jordan Sissel [mailto:notifications@github.com]
Sent: Tuesday, September 04, 2012 9:54 AM
To: logstash/logstash
Cc: Jeremy Einfeld
Subject: Re: [logstash] Correction to GELF forwarding (#194)


In lib/logstash/outputs/gelf.rb:

> @@ -39,6 +39,16 @@ class LogStash::Outputs::Gelf < LogStash::Outputs::Base

>    # is useful if you need to use a value from the event as the facility name.

>    config :facility, :validate => :string, :default => "logstash-gelf"

>

> +  # The GELF version designator. Dynamic values like %{foo} are permitted here.

> +  config :version, :validate => :string, :default => "%{version}"

> +

> +  # The GELF line number. Dynamic values like %{foo} are permitted here, but the

> +  # value should be a number.

> +  config :line, :validate => :string, :default => "%{line}"

> +

> +  # The GELF file. Dynamic values like %{foo} are permitted here.

> +  config :file, :validate => :string, :default => "%{source_path}"

this should probably be %{@source_path} not %{source_path}, right?

—
Reply to this email directly or view it on GitHub<https://github.com/logstash/logstash/pull/194/files#r1524811>.
got it 02a7e1b.
I don't think such implementation details needs to be in readme. If you really want it, just add (use JSON file since options may not fit in environment variable) at the end of first bullet (line 15).
@seanmonstar - https://github.com/visionmedia/ejs/pull/66#issuecomment-8536481

0.8.3 is now published.
@ozten - it looks like it was moved to just below the "bigtent" declaration.
nit: please add an empty line before this CSS declaration. 

Something I forgot and don't know if we talked about already, why don't we need the height? Is it set automatically based on the number of maximum visible rows of the tree?
parenthesis must be on the following line
@vrthra case where package with that name doesn't exist
Any particular reason this scenario doesn't have a `it "restores the hidden constant'` example?
Orphaned code, not used anywhere.
are you sure the HashSet and HashMap operations are concurrency-safe? Ine the case of Array, the assignment is safe when done by multiple threads. Don't think it is advisable to change mutable hashmap concurrently.
I came from C/C++ world, so I love semicolons ;-)
again define, otherwise nice to move it to own separated method
Are not you fixing use statements in tests?
I think that it's possible replace all the logic in this elseif with:

```
$query = $this->getToken()->toQueryString(
    $this->getRequest()->getUri(),
    $this->_config,
    $this->_getSignableParametersAsQueryString()
);
$this->getUri()->setQuery($query);
```
If not description is present in the docblock then return the name of the function
Remove spaces before isset
It's not necessary implement the interface because the parent class already implement the interface
Credo anche io che sia meglio.
Don't forget to remove "use Zend\EventManager\EventsCapableInterface;"
Is there an advantage to having the `$` though?
Replace after rename to `staging` (same line above)
show me where please @stof
typo s/o/of

also, this is a minor thing, but usually we specify the range with [begin, end). but as I said it's a minor thing.
are we still saving the file somewhere?
Again? Change this name :)
Same thing goes here for the redirect. Though if the view helper is called, the redirect param must be set explicitly as the request object is not available.
> Sure, it might be taste, but most web devs from my experience are using my suggested style.

done...

>What do you mean Meteor ships all that stuff?

besides the developer's code, Meteor is shipping a bunch of other js code to the client. unfortunately, neither do i know in detail what Meteor is exactly shipping to client nor do i know in which case ie.js is needed. but i assume the development team of Meteor has taken care of IE's unique features and included whatever is needed... 

based on this assumption, i suggest to remove ie.js...
Usually, yes, but in this case you specify both min and max scale, so it can't really be anything else than 1.0.