labofsahil/github-event-dataset-2012 huggingface dataset:
3ead2ae36caafc0d40059d6512bd4855b182523b is pushed with the change.

----- Jason Bau <notifications@github.com> wrote:
>In main/courses/reports/views.py:
>> @@ -0,0 +1,16 @@
> > +from courses.reports.data_aggregation import *
> > +from django.http import HttpResponse
> > +from django.shortcuts import render_to_response
> > +from django.template import Context, loader
> > +from django.template import RequestContext
> > +from courses.actions import auth_is_course_admin_view_wrapper
> > +
> > +@auth_is_course_admin_view_wrapper
> > +def main(request, course_prefix, course_suffix):
> > +  try:
> > +    common_page_data = get_common_page_data(request, course_prefix, course_suffix)
> > +  except:
>
>that's fine.
>…
>On 9/23/12 1:40 PM, halawa wrote: > > In main/courses/reports/views.py: > > > @@ -0,0 +1,16 @@ > > +from courses.reports.data_aggregation import * > > +from django.http import HttpResponse > > +from django.shortcuts import render_to_response > > +from django.template import Context, loader > > +from django.template import RequestContext > > +from courses.actions import auth_is_course_admin_view_wrapper > > + > > +@auth_is_course_admin_view_wrapper > > +def main(request, course_prefix, course_suffix): > > +  try: > > +    common_page_data = get_common_page_data(request, course_prefix, course_suffix) > > +  except: > Alright. Thanks. I will do the fix so this function uses the > middleware instead. I believe, reading through the middleware, that > you have more specific exception handling. So would that be sufficient? > … <#> > I will send you some results on the execution time for different nums > of students shortly. I am running a db_populate --massive with 4000 > students on my machine and its taking some time to fill the DB. Can I > take the fix for undelayed writing of rows as a P1 and work on it > after the pull request is merged? > ----- Jason Bau <notifications@github.com> wrote:
> In main/courses/reports/views.py: > @@ -0,0 +1,16 @@ > +from
> courses.reports.data_aggregation import * > +from django.http import
> HttpResponse > +from django.shortcuts import render_to_response >
> +from django.template import Context, loader > +from django.template
> import RequestContext > +from courses.actions import
> auth_is_course_admin_view_wrapper > + >
> +@auth_is_course_admin_view_wrapper > +def main(request,
> course_prefix, course_suffix): > + try: > + common_page_data =
> get_common_page_data(request, course_prefix, course_suffix) > +
> except: sorry, i'm going to make a stink about this one. please catch
> only certain types of exceptions, instead of all exceptions. You
> should only catch the exceptions you expect, and can handle, meaning
> you should be minimizing the set rather than maximizing. I am going to
> gate the pull request on this fix. — Reply to this email directly or
> view it on GitHub.
> 
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Stanford-Online/class2go/pull/797/files#r1669409>.
>
>
> 
>
>        —

>        Reply to this email directly or view it on GitHub.
>
>
pretty sure floor is correct.

consider currentTotalActivity=75, lowerBound=50.  this gives desired size of 1.5 which means NO MORE THAN 1.5 MACHINES else we break the lower bound.  (2 machines gives us currentMetric=37.5 < lowerBound.)

unless we fall afoul of the thrashing criterion (upper bound).  if upperBound < 75 the following check will kick in to say whoa, i need 2 machines else i'm too hot.  and i'd rather be too cold than too hot.
Now using a custom CommandExecutionException throughout the code
I'm a big fan of `alwaysRun=true` on pretty much all BeforeMethod and AfterMethods, so that if someone in the future adds a method not in the same group then the setup/teardown will still get run (i.e. same behaviour as in eclipse ide!).
Oui, c'est pour ca que je vais explicitement separer ces activites. C'est bien sur la distribution en magasin qui va s'ecrouler, et celle sur internet qui est en compet avec le P2P
URI of transparent image 20x20
wtf how did that get in there, doh
I have splitted according to test cases. This is for logging. Because
sometimes to look at spec file about 1000 lines could be chalanging.

Sent from my iPhone

On 19.11.2012, at 21:07, Tony Spataro <notifications@github.com> wrote:

In spec/db/cassandra_model_part2_spec.rb:

> +  	it "should display attemps time in milliseconds (ms)" do
> +      first_started_at = 100.0
> +      started_at = 100.0
> +      @logger.should_receive(:debug).with("CassandraModel get, cf=cf_name, keys=1, time=10.0ms")
> +      RightSupport::DB::CassandraModel.do_op_log(first_started_at, started_at, 0, :get, 'cf_name', [11])
> +    end
> +
> +    it "should display total time milliseconds (s)" do
> +      first_started_at = 99.0
> +      started_at = 100.0
> +      @logger.should_receive(:debug).with("CassandraModel get, cf=cf_name, keys=1, time=10.0ms, retries=1, total_time=1010.0ms")
> +      RightSupport::DB::CassandraModel.do_op_log(first_started_at, started_at, 1, :get, 'cf_name', [11])
> +    end
> +
> +  end
> +end
> \ No newline at end of file

Hmm, good point. It would be nice to put these tests into the existing
spec. RSpec rewards you if your spec files are named after the class they
test. (Tools like Autospec and ZenTest work better, for example.)

—
Reply to this email directly or view it on
GitHub<https://github.com/rightscale/right_support/pull/64/files#r2170214>.
Atenção na identação: 4 caracteres para códigos python
Current best-practice is to use `<i>` for icons, e.g. `<i class="icon-twitter"></i>`. Here this would be "more semantic" and you don't need to re-write so many CSS rules, that `<p>` uses.

Also, we could include the text for links, lynx and other text-based browsers here: `<i class="icon-twitter">twitter</i>`
And then just remove the text in CSS using: `@include image-replacement;`
В портфоліо бандлі юзається.
what exactly is this blacklist? should we maybe make this configurable (not in this PR but as a later task)?
Hi! Welcome to Tomahawk :)
Please fix styleissues
these two lines change the object's namespace property. any filter called later than this one will be faced with a wrong namespace property.
On Thu, Feb 23, 2012 at 2:06 PM, Dan Yamins
<reply@reply.github.com>
wrote:
> WEll, it would more accurately reflect the structure of the data somehow;  it would mean that one table would be "job management data" and would not be edited by the bandit.evaluate, and all modifications to it controlled by the bandit algo and Experiment, and the other table would be "results data", with the possibility of having many result records associated to a single job record, which would be written to asynchronously by the bandit.evaluate.
>

Sure, but these reasons are implementation strategies that wouldn't
even be visible at the level of the Ctrl API would they?

Lets talk first about what we need the from the Ctrl object's API to
get new jobs into the database. We already have
ctrl.trials.insert_trial_docs.  How do we construct the trial docs?
Bandit.evaluate gets an empty template from ctrl.trials, fills in the
rest, and inserts it.

I don't see how changing the db layout is going to help get this done.
some wrapping would make the XML easier to read
Sounds like a hack to give back the focus to the field. Can't we use some magic CSS classes to prevent the focus to be stolen?
It might be fun to have just a few fun commands like that. Things like !nvidia would link to this: http://www.eteknix.com/wp-content/uploads/2012/06/LinuxVsNvidia.png (lol). or !help to the wiki (or site).
I don't think that it is a good idea to access the PartsBin via WebResource / Network layer. Why isn't the lively.PartsBin interface used? If the interface does not provide the needed accessors then please extend it. This implementation here is very brittle because it will break as soon as the PartsBin implementation changes, won't use optimizations like caching implemented in the PartsBin layer and is the wron level of abstraction for a "PartsBinFlap".
@desunit & @mythz  I use thread specific storage for this object because the existing code was depending on a static class.  I originally tried the approach used by EndpointHostInstance and found it broke too many of the existing api's.  By having the thread local store we can keep the interface the same as the existing code.
这里我也没有经验，不过我感觉应该不会，应该是和db/migrate/20121030133106_add_attributes_to_events.rb 的 timestamp 保持一致吧
Именно, что вся программа должна быть обёрнута в try..catch. Показывать пользователю stack-trace -- это моветон. Вопрос вот в чём. В этом методе мы выполняем команды какие-то. Они теоретически могут как-нибудь ругнуться. Можно показать пользователю стектрейс и выйти (сейчас так будет), можно показать сообщение об ошибке и выйти/позволить работать дальше. Shell, как мне кажется, не должен завершаться по каждому поводу.
Как насчёт обратного слеша?
We don't actually support prettyfaces, so we need to explicitly call out in a comment that this is an unsupported component.
This section is about library mode I think? Title should call this out, as it does for client-server mode.
True, I'll remove this dependency
Small typo: self._first_address_locator
ありがとうございます。それにします。
Need info: I thought we decided that i was fine to go with pk? If it's easy enough then I like slugs better though.
Logging at WARN in the 5 places seems fine. Guess I was thinking about a helper method in Constants or similar so you'd have access anyway, but that's certainly not the cleanest.
Why is the FIJI updaters page content not just on this page?
@nashby I'm unsure, if we do `:hint_html => {}` we will have the same issue
My fault, sorry. But is_in_mask is identical as far as I can tell?
Yes, `PyDict_Clear` in `Objects/dictobject.c` seems to actually free memory. A `dict` is implemented as an array of keys and an array of values, and calling `clear` will deallocate both. `del term_counts` may be even better, though marginally so.
I'd say "one" but I guess "unity" is ok.
ні. ти маєш перевірити екшн контроллера. роут wallpaper_delete здається
Yes! Pushed.
Define this outside of subscribe. There's no reason to redefine on every subscription.
Does this need `explicit`?
`Object::connect: No such signal QTcpSocket::error() in ../NativeTCPSocketObject.cpp:315`
I think you need to include the parameters: void 	error ( QAbstractSocket::SocketError socketError )
This is a run-time warning, make sure to check the logs.
`|| !args[0]->IsObject()` is not nevessary, type Date is of type Object
Type d'authentification supporté
Cannot see what line you are talking about 

May be it moved because of an update un the PR, i ll search
This warning has no need, since within transactions files will be added and removed while others will be present. This is just a fact of how it's implemented.
What happens in <= 1.37 - where numberOfSlaves is unset (0) ?
Even on attributes, just for consistency - so not have the call present in the switch.
This change set reduces the model tests to a single trivial test.
I think so, I was just concerned about cases where the database may be unnecessarily resynced. Can this be a destructive call if used at the wrong time?
simple, elegant.. I'll update that.
No, I'm referring to the node["network"]["interfaces"][listen_interface] lookup. Doesn't that assume somethign?
yes, it is. The default API's value uses nova-volume. So we are changing the defaults only to disable nova-volume and enable cinder support.
Click on ConfigFile and choose Find all references. VS doesn't find line where var localConfigFile is declared.
Does this or accounts actually use ICU?
cam MSF_LICENSE be used?
I changed the detection to be much shallower, to just check for the API and not whether a context can be created.  That will let me detect IE, at least, though I'll still need to catch exceptions when creating a Scene.
This code is fine as is, or if polyline wants to access `_collection` directly.  I remember when I first added the getter to `BillboardCollection` (which is where I bet this came from) that I was on the fence about it.

The collection property shouldn't be public because end users don't need it - it's an implementation detail that would bloat the interface.

I still think there is some merit in naming conventions for internal-ish members but that discussion can be moved elsewhere - or better yet, delayed :).
then how about using delegation? Do u really have to extending via sort of inheritance?
why not this?

```JavaScript
function VertexDraggingHandler(view, options){
   var delegate = DrawingHandler(view, options);
   var self = {
      dragMove: function(payload) {
         delegate.dragMove(payload);
			// modify the target shape, which is the chassis
			var path = view.target.attrs.path;
			path[options.pathIndex][1] = view.getPosition().x;
			path[options.pathIndex][2] = view.getPosition().y;
			view.target.attr({path: path});
      },
      
      dragEnd: function(payload) {
      		delegate.dragEnd(payload);
			   self.trigger(
			      Events.chassisShapeUpdated, 
			      {shape: view.target}
			   );
      }
   };
   
   $.extend(self, EventHandler(view, options));
}
```
nvm, just saw it below
This hint text only goes away once the field contains a value (Ex: user starts typing text into it). It doesn't disappear based on focus.
This class can't be removed either because of modules
Also true for `touchcancel` on a tableview. May only apply to iOS.
Функция имеет слишком много параметров подумай как можно уменьшить их число.
Do note that not appending the embedded document is okay because the fields referenced by the embedded document are updated in the call to self.process_document above.
Why is this second StringBundler needed?
Line 683 adds the `errorClass` to the label, so this would always be true. Could you address that, along with unit tests?
Ah marketing slogans. :-)
Please document what is being worked around.. specifically, what bug number?
Not necessarily. It takes an argument, so it can be used as a setter (and setters can return like you'd expect from `opAssign`), but the fact that it has a default argument means that you could use it as a getter. I could see an argument though that it doesn't really make sense for it be a getter, since it presumably does some setting as it does take an argument (default or not). So, I don't know what the correct answer is. It doesn't surprise me though if the compiler lets it be used as a getter.
I understand - only thing is I don't see a compelling case for inclusion in the standard library.
I don't think so actually:
-None of the random generators return by ref
-Boost doesn't return by ref
-The return type is just a built in integer/floating point

Unless you mean it as a way to "write to front and modify the payload"? I don't think we should leak this kind of control, but don't have the PRNG knowledge to certify it. I'd rather not be the one to open this functionality

Maybe return by const ref then? Your call.
Alignement is good in the editor, it's just github and its tab management.
Write comment what this function does.
Lift default duration into a constant.
Why? The method is designed to load the categories into the object, returning the object itself is unexpected and might cause problems down the road. The return type of a worker method should always reflect what its doing, so in this case if it would return the category information if it would return anything at all.
Keep in mind that this is an API and one should always be able to use an API without understanding/knowing what is inside it. Returning types/information out of nowhere, doesn't fit this model.
nitpicky!
I don't know the reason why there is a specific boolean for pdf conversion. 
Is it because the previous solution was not optimized enough and may caused issues while developing ?
In that case, I think activating this function will impact only those who want to work on the pdf conversion. The others won't see the difference. Benjamin, what do you think ?
why do they have ids?
maybe this class should be abstract
code looks a lot more clean now that all of the encoding mess is removed
Drop these lines from the pull request please. I'll fix them in an unrelated commit.
We should not be using `.live()` or `.bind()` anymore -- we should only be using `.on()`. This change should be made throughout this pull request.
Then it is going to conflict with basically every existing use of Phobos' Unqual in user code.
(Yes, even though it is not accessible at all. Walter has argued that this is intended behaviour.)
Please use `LPWCH` instead of `const(WCHAR)*`.
Don't include date range fields with the other facet fields
I think more about code convention and if we plan to make the same for others enums, maybe there is any reason to make the name of this method more API'able (;-)): `valueOfIgnoreCase`, `fromStringIgnoreCase` etc...
Artem, the wait only applies to tasks that have started execution. Tasks that are *scheduled* but not yet executing do not affect the wait time.
Suggest we add a comment about how to get this to run in STS...

    -javaagent:/home/<user>/.m2/repository/org/springframework/spring-instrument/3.1.1.RELEASE/spring-instrument-3.1.1.RELEASE.jar
I think this line needs to be optional. We didn't want or need it in our configuration.
ok, i make it.
No prefix.
Ok, I surrender: how can I retrieve the driver's name without injecting anything into PdfPreviewLoader ? (I'm missing those beautiful global variables and classes from S1.4).
Reading directly the configuration file is, obviously, not an option.
Honestly, I was hoping you would know =).

Check out the following examples:

```ruby

require 'socket'

class ConnectionPool
  def initialize(&blk)
    @connection = blk
  end

  def method_missing(method, *args, &blk)
    p "mm: #{method}"
    @connection.__send__(method, *args, &blk)
  end

  private

  def send(*)
    p "called send in ConnectionPool"
    super
  end
end

class MySocketClient < TCPSocket
  # send msg over TCP
  def send(msg)
    p msg
  end
end

client = ConnectionPool.new do
  MySocketClient.new "eventmachine.rubyforge.org", 80
end

client.send "here's some data!"

# "mm: send"
# test.rb:10:in `method_missing': undefined method `here's some data!' for #<Proc:0x007fbf69802d68@test.rb:28> (NoMethodError)
#	from test.rb:32:in `<main>'

```

Here's what happens in em-sync *without* the `def; send(*); super; end;` override (but still using `conn.__send__`):

```ruby

      @socket = EventMachine::Synchrony::ConnectionPool.new(size: 1) do
        EventMachine::Synchrony::TCPSocket.new 'eventmachine.rubyforge.org', 80
      end
      @socket.send("PING").class.should be(Fixnum)
```
```bash
Failures:

  1) EventMachine::Synchrony::TCPSocket should work when wrapped in a connection pool
     Failure/Error: @socket.send("PING").class.should be(Fixnum)
     NoMethodError:
       undefined method 'PING' for #<EventMachine::Synchrony::TCPSocket:0x007ffab1188d38>
     # ./lib/em-synchrony/connection_pool.rb:68:in `block in method_missing'
     # ./lib/em-synchrony/connection_pool.rb:23:in `execute'
     # ./lib/em-synchrony/connection_pool.rb:67:in `method_missing'
     # ./spec/tcpsocket_spec.rb:26:in `block (3 levels) in <top (required)>'
     # ./lib/em-synchrony.rb:26:in `call'
     # ./lib/em-synchrony.rb:26:in `block (2 levels) in synchrony'
```

However, when `def; send(*); super; end;` exists on `EM::Synchrony::ConnectionPool` then all is well.

Now that I'm digging further I see that this **only** works when I privatize `ConnectionPool#send`. What's odd is if I place a `puts` in the method nothing is output to console (meaning it's never invoked???). Yet, when I make the method public I get the same NoMethodError from above.

Got any bright ideas?
We should probably git ignore all js created by coffescript
Perhaps an idea not to show a marker if the count is 0; some projects might not use the markers and it could be distracting for them? What do you think?
sur (manque le r)
Is `:put` out on purpose?
**Warning:** No PHP code was found in this file and short open tags are not allowed by this install of PHP. This file may be using short open tags but PHP does not allow them.
Any particular reason you changed the name?
I'd align those `||`, but that's just a matter of style.
yeah, so you want to see if the user already has an answerset for the questionnaire that you are dealing with, if they do, add your questionanswers to that set, otherwise (as in the case that you are answering the questionnaire for the first time) you create a new answerset
Great! Please push the changes to the same branch (`sandroacoelho:ScalaAhoCorasickSpotter`)!
ToDo: remove tabs
sorry, copy and paste error. 
What I want to say is that I am not sure what you acctually test here. To me it seems that you test the content of an external data source and the Graph&Triple class which are part of Jena and not culturegraph. Test in culturegraph should test culturegraph code and not Jena or the integrity of datasource unrelated to culturegraph.
Internal use only/autogenerated, should not be exposed.
Here's that bug I was talking about. Pipe character is a regexp special character. Needs to be escaped to match.
Typo: server.
In de praktijk is de token dus soms wel leeg en soms niet, dan is die check inderdaad nodig
Kyllä pitää `reduce`:a käyttää. Ei pisteitä.
We will have to add the XQJ classes/interfaces jar as a separate dependency to make this dependency optional. Currently this jar contains these classed too and hence compilation will fail when we make this optional.

I'v added sedna's dependency  as optional and included a test for XQueryExecutor using the sedna's provided datasource. However there is  one jar xqj2-0.0.1.jar which is missing in our repo. Need to add that too and add it as a dependency in the sedna xqj's pom. The archive downloaded from sedna's site contains three jars. We basically need all three of them to be present in the repo (one is already there). After these two are added, i'll make saxon's dependency optional.
Good catch, I forgot a statement
All annotations should be at the implementation.
Fixed in 83910050
en revue, manque un e.