labofsahil/github-event-dataset-2012 huggingface dataset:
Ajan sai puolitettua jo sillä, että se on from tilausrivi join lasku. Mutta sekin oli vielä melko hidasta.
this structure would probably be more clear if it was: 

indicators [
 { "key": "foo", "expected": "---"}
 ...
]
I like this common vote method for both of the vote actions.
If the purpose is to quickly recover from a scenario, can we use try/catch instead?
In general, exceptions should not be used for control flow. :)
maybe better in `default_settings.yml` and then use the ConfigResolver ?
-1 for "standalone" {@inheritdoc} tags. Line noise (as in: makes reading the actual code harder, and for the generated doc, well, let's improve doc generators instead)
same remark as for group
maybe make this `jboss-as-changeme` or something?
新しめのhamlでsyntax errorの様子でした。
This indent was correct. It's inside the `<li>`.
Should be indented in.
Since you're deriving an object that contains the "previous" properties and values, let's change this to `previousOptions`—I believe this will make following the code logic a little easier.
It uses GridStyleConfigurator as well? That did not show up in the screen shot?
`Zotero.Utilities.cleanAuthor(author.textContent, "author")` here is identical to `var newAuthor = Zotero.Utilities.cleanAuthor(author.textContent, "author");` You don't need to call `Zotero.Utilities.cleanAuthor` again, the result is already in `newAuthor`
Hmm... I have no idea. Think it was a change where I attempted to remove one of the parameters, but ended up changing it back (and doing that manually).
ok, externalProjection is fine, although I'm thinking it's important that mapping is suggested somehow. Just "externalProjection" seems to be too little somehow.
This string must be absolute and also not end with a slash.
Now that I'm not looking at this on my phone... I think there should be an assertion on this for the return of #add_source. I'd rather use #stubs instead of #should_receive and assert the return as that frees the test from assumptions of implementation and makes a more explicit test. 

```ruby
subject.add_source(source, false).should == [CookbookSource.new(...)]
```
yep my bad. I fixed it.
Looks quite bad, if there's no UUID on the upstream distributor, something is really really wrong. Seems like we shouldn't be filling one in quietly.
i18n_domain="eea"
Oh in the base class. Got it.
чем сокращать слова лучше сократить их количество.
нет. Graph вообще не должен знать ни об одной реализации вершин на мой взгляд.
Should this require `middleman` rather than `middleman-core` to preserve behavior?
Your probably right, was thinking about reloading, but that doesn't actually make any sense :-p
Derp, will do.
enumeration should be complete - http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html
Preferable to just let this be integers no? qhttpserver users will then only be as spec compliant as they choose to be, rather than having compliance forced.
jhgjhjh
This could use `->>` and `maptrans`, but it doesn't really matter.
I'm going to update this to point to `https://github.com/rwldrn/idiomatic.js/tree/master/translations/zh_CN`
These should return a 400 - Bad request, rather than a 500 as it is not really a server error.
Too many spaces here. Should only be 2.
I don't think any of these new imports are needed here anymore.
The abstracted isAdmin function is going to make this more fun to read. And fun is a good thing.
What does this mean? Should there be a TODO here ?
You should check for an error coming back from the Cassandra query first. When that is negative, you can do Pubsub.publish.
According to our spec, we can have multiple config files per module. This function should get the list of files inside of the config directory and should load all of those into separate sections in the UI.
Would be good to have a one-line scaladoc for this method. The return type `AnyRef` does not say much..
I'm wondering if the performance gain this is worth the loss of precision. Did you test this in some big file, e.g., Typer.scala, and noticed better responsiveness.
Is the false condition needed? $contains is already set to false by default. It makes it a bit clearer, but it's an unnecessary conditional being checked.
which does exactly the same, only that isenabled() is not a static function right now.
set it higher,  > 254.0f/255.0f
this wont be the right dirname, need to `path.dirname(file)`
Why would summaries by null when you "new" it up above
- (void) mutate:(int)percent; // тут переводим в кол-во запускаем mutateByCount
- (void) mutateByCount:(int)count;

а в mutateByCount  запоминаем рандомный индекс и итем, меняем его на уникальный, удаляем из массива, запускаем mutateByCount  count-1, пока не будет последний элемент. Будет так же коротко, никаких циклов.

Расчет занятости стека, я не знаю как на OS X, но если взять самую старую программу под win32 с размером стека 16кб, то будет можно сделать 2048 ( 4 байта на возврат, 4 байта под указатель на массив) запусков, для нашей задачи достаточно.
Так вот же, в CellMutate.m:

    if (percent > 100 && percent <= 0) return; // Если передали 0, то делать ничего не нужно.
    int qnt = DNALength * percent / 100; // Количество элементов для замены
Крайне рекомендую просмотреть 3 лекцию, если вы этого ещё не сделали, и заменить этот метод на (NSString*) description
Понял мысль, спасибо за наводку. Сделал readonly, так как синтаксически пока не знаю, как правильно копировать ссылки.
Не из-за этого, а потому что для описания нуклеиновых кислот лучше подходит тип enum и к тому же NSMutableArray оперирует ссылками на объекты
согласен. исправлю
https://github.com/SpoutDev/SpoutPlugin/blob/master/src/main/java/org/getspout/spoutapi/material/item/GenericTool.java#L37
You test that it has a name but you never output the description and therefore never actually test that description is ever saved.  This needs to be tested somewhere.
Since the module is already ``Transport`` might be clearer if this class was ``BaseTransport`` or ``TransportBase`` or something.
FWIW, I agree with @coderanger. The current implementation doesn't work for me, because I do want the middleware at the front, and I want to to use a custom logger:

```ruby
module Sidekiq::Middleware
  module Server
    class Raven
      def call(worker, msg, queue)
        ::Raven.capture :logger => 'background' do
          yield
        end
      end
    end
  end
end

Sidekiq.configure_server do |config|
  config.server_middleware do |chain|
    chain.add Sidekiq::Middleware::Server::Raven
    # others
  end
end
```

If I update to your code, I'll need to manually remove the `Raven::Sidekiq` middleware.
Immediately below here, we should assert that `wasDisposed` is still false.
Not a big fan of this name, but I'm not sure I have a better suggestion. Something including the word "pairs" might be good, since we're basically talking about a higher-order function that operates on every consecutive two items of the stream.
But anyways, thanks, it just unveiled a bug, forgot to assign after
indexing in line 520 -- thanks!


On Thu, Dec 6, 2012 at 5:34 PM, Denis-Alexander Engemann <
denis.engemann@gmail.com> wrote:

> Why do you want t index before / during copying? Why isn't it sufficient
> to copy and then do the indexing / overwriting as is now?
>
>
> On Thu, Dec 6, 2012 at 5:31 PM, Christian Brodbeck <
> notifications@github.com> wrote:
>
>> In mne/epochs.py:
>>
>> > @@ -463,19 +493,30 @@ def __repr__(self):
>> >      def __getitem__(self, key):
>> >          """Return an Epochs object with a subset of epochs
>> >          """
>> > -        if not self._bad_dropped:
>> > -            warnings.warn("Bad epochs have not been dropped, indexing will be "
>> > -                          "inaccurate. Use drop_bad_epochs() or preload=True")
>> > -
>> > -        epochs = cp.copy(self)  # XXX : should use deepcopy but breaks ...
>> > -        epochs.events = np.atleast_2d(self.events[key])
>> > +        epochs = self.copy()
>>
>> _data is private so nobody is supposed to modify it, I would just use
>> numpy indexing , i.e.
>>
>> if preload:
>>     new._data = self._data[index]
>>
>>  —
>> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/229/files#r2337217>.
>>
>>
>
>
We should factor this out in to their own module. Importing this 'causes all this `program` junk to run.
Please use if(err) instead
6pm should be 18 not 17
Changed. I removed isPriorDateOfDeathEstimated() method also.
This is useless as the previous paragraph ends with ``::`` which means creating a code-block for the default language (which is of course configured to be PHP when rendering the symfony doc)
Awesome, done!
added extension setting for akka.amqp.timeout.
Could add here something like "(default). Chooses between 'D' and 'FW' based on number of nodes and edges.".
oh, right. well, we'll need to come back to this one at some point...
`manifest_path` is escaped here
This problem appears to be restricted to Webkit, and may be a bug. [The spec](http://www.w3.org/TR/css3-values/#rem-unit) says that 1rem is equal to the computed value of `font-size` on the root element, except for `font-size` on the root itself (in which case 1rem is equal to the initial value). However, in Webkit it appears that `line-height` on the root element also uses the initial value rather than the computed value. One possible work-around could be to set line-height on the body while setting font-size on html. Another could be to set line-height in ems in `establish-baseline` only, even when `font-unit` is `rem`. I need to investigate further, and also file a Webkit bug.
I don't want to have two DSL's for assertions. It makes understanding and writing tests as a new dev much harder.  If we were to switch to a different DSL I would strongly prefer rspec. 

I don't remember exactly why, but something in shoulda cost me a whole week of work when I was at Gowalla :cry: 

These days i'm trying to use rails to build rails as much as possible, and turn pain points into pull requests, hence why I started out without factory girl or rspec. I see the value in contexts though, wish they were in test unit...
Could this go to the models?
the spec says that so we roll w/ it :P
They're at the bottom...
`toWorkdingDirectory` → `toWorkingDirectory`

Can you please also move the `error` parameter before the blocks? Convention is that blocks always come last in parameter lists, because it reads better if they're provided inline.
Ups. Why has this slipped through again? I'm sure I have changed this! Lets do it again.
This should be more like profile_args.setdefault('preferences', {})['extensions.jsbridge.port']  = jsbridge_port # or just jsbridge.find_port() for that matter
This step fails:

E AttributeError: 'unicode' object has no attribute 'text'
tests/test_smoke_tests.py:36: AttributeError
'console.log'  should be removed (line #403)
Ticket (and README.md) state that the mirrorlist should be the url attribute.

```
mirrorlist=<%= @url %>
```
Note to self - check whether this still works with / without the branding widget
I'd like to keep this around and at least use it internally in this file. That'll make switching to the new way more straightforward.
This function's signature forces to create these codes
please don't use camelcase - but create_diagonal_matrix as name
please use { }  to mark the code block in this for loop too for increased readability
Wasn't this replaced somewhere with whiteblacklist.yml ?
hmmm what's the difference between `active?` and `enabled?`
No, sorry, good catch.  I'll revert this.
I found the previous menu structure confusing.  This split was kind of there, but these sections were under "Additional Notes", which is like filing under "miscellaneous".  I don't much like "Administering" either, and agree that signals probably fits better in the programming section.

Maybe "Managing a WiredTiger Database" for the remaining sections on files, directories and configuring / tuning?
Should we document this?
Because the old test case does not work anymore: 2 is a valid name now. I could have changed it to check whether that's recognized correctly, I guess, but test for that exist.
Is it possible to define a more specific return type; maybe using a union like ``null|Object``, or ``string|array``? Please also check line 225.
same here, or we can just check the range and assign, something like:

```c
 enum {
   DATA_IMAGE = 1,
   DATA_MIME,
   DATA_IMAGE_AND_MIME
 } data_mode;


..

if (mode >= DATA_IMAGE && mode <= DATA_IMAGE_AND_MIME)
```

otherwise throw or noop, something like that
Actually I wanted to solve like such problem - for my BB emulator screen height - width are  are 1024-768 but for device usually it wold be something like 320 - 480. Just tried to distinguish somehow emulator from real device, but now changed the code like following
 var isBlackberry = Titanium.Platform.name === 'blackberry';
	var scaleX = 1;
	var scaleY = 1;
	if(isBlackberry)
	{
		scaleX += 1;
		scaleY += 2;
	}
the scaling problem occurs because if we set button width for example 200 - it is cut in the middle and not shown wholly (at least on the emulator it is so)
"implements Runnable" is encouraged over extends Thread.
Will pagelets work okay in mysql without these migrations in 0005 and 0006?
No.... `value !== undefined && ...` is still there, so the existence of the value parameter would be what decides whether it's a get or a set.

However, we'd have silent fail with `$.cookie('foo',{bar:42})` in old browsers. Can easily throw an error for that case though.
This is the important change. I was getting `{["US", "MI", "LANSING"]}`. This changes it to `{"country": "US", "state": "MI", "city": "LANSING"}`.
Should we call this twice (line 2)
I did it so this line would be visible, howerver i am expecting changes to be asked for, for both the content of the mail and its subject.
Since `normalise.css` is a drop-in css thing, we shouldn't be changing it. Not a big deal here, since this is a whitespace diff, but I think it should still get reverted.
Which URL?
*e-mail address*?
No, you should make pull request to work as it is. For things which are not yet in the pull request, you can add TODO or new ticket. But for things which are in the pull request, you should make it so that after it is pulled, things work. And this does not work for nobody else, except if by luck it has post with exactly this ID.
I really don't like `description` name here. Could you find some other name for this value?
This looks like it is still under heavy development, any reason not to subclass `dict`?
Title should be "404 - not found"
This is wrong, imo.

As you can read here: http://www.wowwiki.com/Dalaran_Arena

>Hotfix (2009-03-02): "We have modified the waterspout effect in the Dalaran Sewers Arena to give players more warning before the water column is fully activated. This should give you a chance to get out of the way or at least let you cancel a cast since you will now know that the water is coming soon. The effect starts up about 5 seconds before the waterspout is fully active. We have a new zone wide sound attached to the effect now also." 

So at least that warning should last 5 secs.
I strongly recommend using [virualenvwrapper](virtualenvwrapper.readthedocs.org). It, by default, puts all created virtualenvs in `~/.virtualenvs` and gives you really great helper commands like `workon`. When I go to work on chipy, I'll type `workon chipy` and it will switch over to using that virtualenv.
Still singleton?
I cannot see the whole line, but this is basically just an other template? Why do you have to define it inline?
Why the equality check? You're merely updating a class member, which is inexpensive. I would remove the equality check: It is wrong anyway to do a string comparison for paths (For such things, you would use QDir::operator==()).
You can create a file called secrets.py, and put those two global strings in it instead of commenting out and committing these imports.
I changed to this format for all functions
"are" again
.... or be configurable ...


On Thu, Nov 15, 2012 at 11:43 AM, Michael Landis
<mlandis@5amsolutions.com>wrote:

> those need to come out.
>
>
> On Thu, Nov 15, 2012 at 11:42 AM, msweaver <notifications@github.com>wrote:
>
>> In
>> Product/Production/Services/DirectCore/src/main/java/gov/hhs/fha/nhinc/direct/DirectMailClient.java:
>>
>> > @@ -155,7 +155,10 @@ public void sendMdn(Address recipient, MessageProcessResult result) {
>> >      public int handleMessages(MessageHandler handler) {
>> >
>> >          Session session = getMailSession();
>> > -
>> > +        session.setDebug(true);
>> > +        session.setDebugOut(System.out);
>>
>> should these be here?
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/CONNECT-Solution/CONNECT/pull/174/files#r2140962>.
>>
>>
>
>
>
> --
> Michael Landis
> Software Engineer
> www.5amsolutions.com | Software for Life(TM)
> phone: 866.526.6042 x774
> email: mlandis@5amsolutions.com
> skype: mlandis5am
>
>


-- 
Michael Landis
Software Engineer
www.5amsolutions.com | Software for Life(TM)
phone: 866.526.6042 x774
email: mlandis@5amsolutions.com
skype: mlandis5am
I just pushed a version in which I made an attempt at this.  In particular, I defined ```HSMError```, which gets used at the top-level C++ classes ```FindAdaptiveMomView``` and ```EstimateShearHSMView``` as a way of collecting the various exceptions that the functions they call.  ```HSMError``` is not wrapped, instead I used the syntax Jim suggests here in galsim/psfcorr.py.  I had to make a slight modification for the ```strict==False``` case, because in that case the ```HSMShapeData``` object that the python-wrapped functions was supposed to return is not defined, so I initialized a new one and gave it an error message.  This means that (1) we have an exception class at the C++ level, (2) exceptions are announced once rather than 2x at the python level if ```strict==True```, and exceptions are not announced at all if ```strict==False``` (but, the ```HSMShapeData``` object that was returned will have nothing except for an error message).  Sound good?
I can add that warning, but FYI this was also the case previously.  Any InterpolantXY was accepted and used without checking if its `i1d` is a `Linear` instance.  I just added the functionality of not requiring the user to hand-wrap a 1-d Interpolant into an InterpolantXY.
again, sorry :)
Thank you, please push those changes since you're already done them, no need to revert.
The numpy version is still significantly faster, because numpy arrays are less flexible data structures than Python lists: every element will be the same type and thus size in memory, and this of course means there is scope for greater optimization.  On my old 10.5.8 laptop the following script:
```
import numpy
import time

ntiles = 3000

t1 = time.time()

ix = [ k / ntiles for k in range(ntiles**2) ]
iy = [ k % ntiles for k in range(ntiles**2) ]

t2 = time.time()

iy, ix = numpy.meshgrid(numpy.arange(ntiles, dtype=int), numpy.arange(ntiles, dtype=int))
iy = (iy.flatten()).tolist()
ix = (ix.flatten()).tolist()

t3 = time.time()

print t2 - t1
print t3 - t2
```
produced this output:
```
6.04220986366
1.11686515808
```
For reference then, if we do note that a faster implementation is available, I vote we go for the numpy!  (For info, testing the original nested `for` loops gave me a time of 5.97471308708, so similar to the 2 line list comprehension above...)
Indeed, parallel would be really nice!
I don't know enough about the XML spec.  Is it valid to have another opening xml tag here?
This was temporary code and has been removed in the newer codebase which will be given in a future pull request so we can leave it as it is for now.
I know - I've read the page. Actually pretty excited about it (and just donated since you obviously know what you're doing). Happy to give any feedback/thoughts/help/ideas to make it (more) awesome.
These `uint`s should probably be `size_t` to be 64bit compatible.
It's cleaner to convert all your migrations into one as part of the pull request.  No big deal though - I can do that as part of merging.
What about re-using the |this.moved| property?
Yeah the bugzilla bug referenced in this pull request has the other half of the fix.
Ah, good call.  Should we be using io.open in utils.py3compat, and just use that?
Instead of doing that on every items of the conversation, can't we do it only once on the container?
ELCHANGES?  Looks like commented code.  Go ahead and just delete it :)  No need to leave your name - git blame covers that :)
Absolutely not a blocker. Just something to start thinking about how we can fix it.
Наличие поля StringFormatter делает его не thread safe.
Происходит исключение здесь, и FileReader окажется открытым.
Пакет не соответствует директории.
Well, the fact that PHP even has a `_createPath` shows that PHP has grown past the initial port. I suppose that you could check with your new unit tests, using https://github.com/WindowsAzure/azure-sdk-for-php/issues/473 as a starting point. Or not; I can always file more bugs later.
you  have to add it
What about when PHP56 comes out! And when I want to `brew install php44`?
This is just a page region (rather than an outright class) so it should not need to return EditListing.
Actually this whole subclass should be indented to make it form a subclass of EditApp, as Media is only accessible via the EditApp page and inherits from EditApp. So Media can't be instantiated on its own. 

We might tidy this up in a separate pull.
Nit: PayPal
this is commented because of a selenium issue in investigation
Just to be clear, this filters out any equal-valued peaks within the same window, right? I thought when we discussed this in the hough-peaks PR, we agreed it was best to leave equal-valued peaks within the same window (as you defined it, a "soft threshold")?
haha stressful but much appreciated :) nothing like the crack of a whip to speed up productivity! haha nah, it's great - I learn quicker this way! :smile:
you could use ``sklearn.utils.atleast2d_or_csr`` instead; this would also make shure that X is 2d
``text.TfidfTransformer`` should make this clickable.
you can use cpdef on this function as it's private
ok, pending cleanup. We need to decide if we want to propagate state info for all buttons or not. If we do, we should consider how best to do this and not be constrained by the old .which and .button, which are somewhat arcane.
What about moving this check into `peer.Header::__construct()`? It'd be safer there. I wouldn't expect a method named `isPossibleXPClassname()` throw an exception.
Excerpt of email to Victor:
"I find the intention and use of the 'GPX file...''' and "Favourites" options in the "Define views" menu rather confusing. When a GPX file is loaded using the 'GPX file...''', the points are loaded as favourite points and which can only be displayed by turning on 'Favourites'. Turning on the 'GPX file''' actually clears the favourites loaded, hence if the user returns without loading a new gpx file, the favourites originally loaded are no longer available.
It is not clear whether the 'GPX file'' map menu option should be a check button or just a button to open the file selection menu.

Is the intention that the gpx files only contain favourites? Are favaourites and historical tracks considered to be the same thing?"

As you indicated, I expect that it is desirable to be able to show favourites as well as the manual track - it makes sense to me.
Please initialize the values from `enum N_TEXT_ALIGNMENT` values
Yeah that sounds like the right move.
Da fehlt 'n Ab-stand, ne?!
That will happen, the 1st time it reads a delta that proceeds this change.
Which is ok, in that case, the above condition is not met, and will nor perform the checks.

As a side effect it will also not perform the crc check, that 1st time also.
I was going to comment on that effect, but seems you have a good comment already. ;)
So much for uncontroversial :)

Why not? Doesn't that make it pretty useless for debugging in a local dev environment?

Why shouldn't we compress it along with everything else?
Just don't use `settings.SITE_URL` at all. It's for the browserid stuff. 
If you desperately need the root url then use the `request` object like I showed you in IRC.
How about naming these functions `emit_true_load_finished` and `emit_false_load_finished`, and deleting the comments.
Or you could use strlen($version)-4 === strrpos($version, '-dev', -4)
I think you should rename this method to `collectParameters`
But doesn't it make sense to use the full function signature since it's known ahead of time? And to get ``message.message_id`` rather than from ``sender``?
Je pense que ta description n'est pas terminé, il manque un quote a la fin du hint.
`(or (apply <= a-seq) (apply >= a-seq))` returns a boolean by it self.
We should add a TODO about matching the other `Form` constructors.
Missing ';' at the end
I dont think this block is needed in your case, is it?
Curly braces =)
Good explanation, I couldn't have done better :-) But I also don't yet get what the `nuS` stands for? :-)
I think in the current form it should be pretty straight forward to do it in 3d. Not sure what I'll have to optimize, though.
`l_x, l_y, l_z = data.shape[:3]` ?
same question here, why hardcoding 100, 100 ? this could be done upstream
Did you make these changes?  Or is this a bad merge?
What is this doing?  Converting a translation (`position`) and a quaternion to a 4x4 matrix?  Instead, let's use `new Matrix4(orientation.toRotationMatrix(), position)`.
We should remove this exception.  We should also allow creating polylines with no positions - if we do not already.  This current Polyline doesn't throw an exception here.  The Polyline in Insight3D use to throw exceptions if less than two positions were provided, but people complained about it.  They wanted to be able to create and modify objects even if they didn't know the positions.
`--mode=<mode>`
`an HTTPClient`
why do we need this line here?
Should be using our `TAG` here instead of `"IntentProxy"`, and our `Log` class instead of Android's. Also, for exceptions, we pass the full exception as the third argument so that we get a full stack trace instead of just the message...

```
Log.e(TAG, "getBlobExtra(): " + e.getMessage(), e);
```
I will also submit that as a bug against HTML5 Rocks.
Fixed in 8f86c335160884754656cc08b0968da42b182d9a
Think this can be simplified using map.  Something like:

      def merge_attributes(cloned_attributes, override_attributes)
        cloned_attributes.map do |clone|
          key = clone.keys.first
          override = override_attributes.find { |o| o.has_key? key }
          override ? override : clone
        end
      end

@thbishop @rickmendes thoughts?
Don't do this. Use the glance-api.conf as shipped by Glance, and add a [paste_deploy] section specifying the flavor according to the auth type. For non-keystone, leave it blank. For keystone, set it to "keystone". This chooses a suitable pipeline from inside of Glance.
Also, it looks like not may functions do this `require`, so I may be incorrect about this being necessary.  It may just be optional.
You could just use `.present?` here rather than negating `.nil?`
может в css уберем %-)
название функции тоже конечно мислидинг
не знаю точно про ссылки, но с указателями dynamic_cast ведет себя спокойнее, он возвращает 0 указатель. Это предпочтительнее, чем ловить исключение, так как бросание исключение очень долгий и сложный механизм.
No, there's a `private:` above.

Labeled attributes are *evil*.
Nit: `s/According/according/`
For me in my incoming pull from my self-explaining `finally-fix-emplace` branch.
Move this to config/initializers/config.rb. You'll see a few examples of how I am loading config files and sticking them on AppConfig
any reason not to point at dart.js here?
Oh, correct. Forgot about that. We could move that into a constructor then.

On Thu, Aug 2, 2012 at 4:15 PM, Pete Muir <
reply@reply.github.com
> wrote:

> > +import javax.xml.parsers.DocumentBuilderFactory;
> > +import javax.xml.parsers.ParserConfigurationException;
> > +
> > +import org.jboss.as.quickstart.cdi.extension.model.Creature;
> > +import org.w3c.dom.Document;
> > +import org.w3c.dom.Element;
> > +import org.xml.sax.SAXException;
> > +
> > +/**
> > + * A simple CDI Portable Extension to "inject" values from XML into the
> instances of a bean.
> > + */
> > +public class CreatureExtension implements Extension {
> > +    private static Document document;
> > +    private final Logger log =
> Logger.getLogger(CreatureExtension.class.getName());
> > +
> > +    static {
>
> well the extension is app scoped anyway.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jboss-jdf/jboss-as-quickstart/pull/324/files#r1297985
>



-- 
Jason Porter
http://lightguard-jp.blogspot.com
http://twitter.com/lightguardjp

Software Engineer
Open Source Advocate
Author of Seam Catch - Next Generation Java Exception Handling

PGP key id: 926CCFF5
PGP key available at: keyserver.net, pgp.mit.edu
This should be changed to simply <pre>require_once 'autoload.php'</pre>
I assume all those `else if` make the implicit assumption that `state == 2`?
please remove the "@Param" part
needs some ws tweaking
Something like 
```php
public function __construct(RouterInterface $router, AllowedHttpMethodsCacheWarmer $cacheWarmer, $kernelCacheDir)
```

?
That makes sense.  Could you add a fixture for the HTML file as well so that we don't make real web requests?
This seems to be a bit early, should be after the ``loadMetadataForClass`` call below.
This will raise an index exception rather than a not found error as the yaml specifies. Can you put this inside an try except for index error and throw not found?
Using a hard-coded width to call `editPosition` seems inflexible. Can we modify how we do that?
This dictionary actually appears to map a suite name to a dictionary of extra information about the suite. It doesn't map cases to suites, which is what the comment above claims. Which is it actually supposed to be?
I decided to perform the empty check outside of the cache::load() function for the command line config file to stop the warning message that the file does not exist from being emitted, as the user has specifically choosen not to provide one, whereas the app/global config file not exisitng is a valid warning.
The proper way to make this always fail is:  Assert.fail("This failed because of X, Y, and Z");
Yep. fixing
Can you me more specific?
Not sure I understand this: does it add the result of running `block` to the query cache of all connections?
It's exactly what we said just a few minutes ago with Maks3w on IRC. I will
propose the changes ;)
--
Mickaël Perraud
Website | http://mikaelkael.fr
Twitter | http://twitter.com/mikaelkael
PGP key | http://mikaelkael.fr/mikaelkael.asc



2012/8/9 Jonathan Maron <notifications@github.com>

> In docs/languages/en/index.rst:
>
> > @@ -471,6 +472,13 @@ Zend\\XmlRpc
> >          * :doc:`modules/zend.xmlrpc.client`
> >          * :doc:`modules/zend.xmlrpc.server`
> >
> > +.. _zendservice:
> > +
> > +ZendService
>
> Could I kindly ask you to edit the document as you see fit. Unfortunately,
> I do not understand your suggestions for improvement. I think you only have
> to edit a couple lines. Thank you very for much for your assistance. ;-)
> Jonathan
> … <#1390cbb7e026aaa8_>
> On Aug 9, 2012 7:34 PM, "Maks" <notifications@github.com> wrote: In
> docs/languages/en/index.rst: > @@ -471,6 +472,13 @@ Zend\\XmlRpc > *
> :doc:`modules/zend.xmlrpc.client` > * :doc:`modules/zend.xmlrpc.server` > >
> +.. _zendservice: > + > +ZendService Probably ZendService should have his
> own index or something — Reply to this email directly or view it on GitHub<
> https://github.com/zendframework/zf2-documentation/pull/139/files#r1344656>.
>
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/zendframework/zf2-documentation/pull/139/files#r1345545>.
>
>
Sort が必要なのは、
MlTitle, Status, ArchiveType, CreatedAt, UpdatedAt 位で良いのでは。
MLPSBy ってよく分からないです。
ほかの省略形と同じように MLPSortBy で良いのでは。
Yes - that's on the TODO list - in fact, many sections have content duplicated or split between pages and will need to be reorganised. Just trying to sort out hierarchy and order / placement just now.
need null pointer check
Making things public is more things to doc, maintain, etc.  I don't expect
these to be widely used, so I would save us the work - and opportunity cost
for other things we could do.  We can always make them public later.
On Aug 6, 2012 11:17 AM, "Kevin Ring" <
reply@reply.github.com>
wrote:

> > @@ -0,0 +1,341 @@
> > +﻿using System;
> > +using System.Collections.Generic;
> > +using System.Linq;
> > +using CesiumLanguageWriter;
> > +
> > +namespace GeometricComputations
> > +{
> > +    public static class PolygonAlgorithms
>
> You can test internal stuff by adding an `InternalsVisibleTo` attribute to
> the assembly.  See Core/CustomAssemblyAttributes.cs in any Components
> branch for an example.  Note that for this to work, both assemblies must be
> signed, which ours currently are not (but arguably should be).
>
> But at a glance, most of these methods seem reasonable to make public to
> me.  What's the concern?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/AnalyticalGraphicsInc/czml-writer/pull/16/files#r1314158
>
yeah when i read this my mind shifted the try:...finally out one indentation, so i thought it was outside set_flow_control rather than inside it
Response:  Opening a window rather than a tab is blocked by Chrome's popup blocker.  This is the only solution I found that allows opening up a tab in the callback.
不可直接显示原始页面。要过滤一下，做如下事情

1. 把所有图片链接改到本地
2. 把文件路径删除
3. top里显示用户头像、姓名，以及确认抄袭按钮（这个显示的时候做，不可html写入数据库）
用download_file_content()下载页面
depending on what the "updated test suite" is :) It worked this morning at least
you should remove this file as you are only redefining variables defined in dependencies/env.sh
We should probably rename these `window.navigator` calls to just `navigator` for consistency.
Можт быть надо цикл?
Try using list of Roles
An insert alias is ok. Let me know when you have made that change.
And AFAIK, Bags of Words means only 1-gram by default.
I think we have three spin buttons, one for hour, one for minute and one for AM/PM, thus the role should be on the the ol
Here's your problem. You just need to get the value of #currency. Not sure what the second line here is supposed to do, but your `var e=document.getElementById("currency").value;` is enough.
Ah, no it's not. It should be a problem in a system library.
Typo: blackberry
Fixed in the next update.
Say what?

No idea here. Are you saying that you can individually enable or disable each of the following logging levels?  Or that setting debug to true or false enables or disables all of these log levels at once?

Also, these appear to be statics, not constants. Or by constant, do you mean Config.DEBUG? Because that won't show up in the JavaDoc here--LOGD, LOGV, and DEBUG will.

Perhaps this should say something like:
This static flag controls whether <foo> log messages are displayed. Defaults to the value of the Config.DEBUG flag. Config.DEBUG is set to the value of the 'debug' property in 'tiapp.xml', or false if the 'debug' property is missing.
`Compose`
This is true for ios5.1+ which is where we want to use this plugin.
GIven your comment below about moving the version checks inside the plugin, I agree with you, so I can move the checks inside to make this more explicit&bulletproof.
Yeah, perhaps the test should intentionally use something different 1) to throw another wrench into the test and 2) to make it clear why it is needed.
Was probably calling some functions only in fsext but not fs? If we are no longer calling functions in fsext, it can be changed to fs.
One action and one *mime*-type is more clear.
JTA code is in provided scope, not sure if I got your concerns correct here, but client side is not relying on JTA being on the classpath at all, but it's assumed that container provides it.
Is this require custom tailored for webworks? Is the unmodified almond not good?
I changed this to FastWriter, StyledWriter is for a human readable format, but we don't really need that for APIs
well, if you want to allow both 2.2 and 2.3, use the proper range (which is the range I used when writing the symfony constraint so a copy-paste would have worked): ``>=2.2,<2.4-dev``.
Using the constraint you just wrote in your comment would be the same than using ``2.2.*`` but with a longer way to write it (which is how composer parses it internally)
Looking at symfony code (in cache from core and bundles), this seems the be the standard way of testing for the request method.

the getMethod() always returns an uppercase string http://api.symfony.com/2.0/Symfony/Component/HttpFoundation/Request.html#method_getMethod 
And the doc sugggest methods in symfony are called "GET", "POST", ...  http://symfony.com/doc/current/book/http_fundamentals.html

NB: There is also a matchMethod() of the RequestMatcher object, but I have never seen it used.
http://api.symfony.com/2.0/Symfony/Component/HttpFoundation/RequestMatcher.html
Would it benefit people if we added a line like the following

    // (results.indexOf(message) !== 0) TRUE
id1, can't this just be id?
I guess this is the todo?
Some white space made it in here.
Actually some data providers aren't prepackaging anything. Better to say just `from data provider`.