labofsahil/github-event-dataset-2012 huggingface dataset:
I'm not sure if the `cpulist` file would contain more than one '\d+-\d+' string...

And the string you want to get is the numbers before/after '-', so I think `(\d+)-(\d+)` could be better, it makes re.findall func returns a list of tuple, then you don't need to call split() every time.
Won't 'list_to_existing_atom/1' do here? If the module is loaded then it will exist as an atom and any crash/error from a bad Op name will be closer to the cause (here, not line 22)
First, I'll admit I haven't thought about i18n yet, beyond "we should support that".

I assume you mean the columns' contents?  I don't really see why that would be localized -- the string comes from the user, and goes back to the user unchanged.  Are you suggesting that we support something like localizing a user-defined builder named "lion debug" to "الأسد التصحيح"?  That sounds hard, requires users to keep their l10n's up to date (and a corresponding UI for such), and seems like it would add more confusion than it subtracts from a buildbot install.

I'd like to support client-side l10n of Buildbot-generated strings -- most of the web UI, at least.  If possible, it'd be neat to do the same with log chunks, so the Buildbot-generated log chunks (e.g., "Process killed after %d seconds of inactivity") can be translated.
Ok I think I might have something. Something like this: `(\()?(\[)?(\{)?...`. and then later in the regexp, I check whether that group (by number) was captured. Let me see if it works.
Advanced developers will not necessarily replace any .vcxproj or .sln
files, myself included. Many of us use a custom .sln file that is located
in a custom directory tree (sometimes on a completely different network)
and reference the .vcxproj files located in the cocos2dx github repo. So
the cocos2dx.vcxproj, cocosDenshion.vcxproj, libExtension.vcxproj, and
mygame.vcxprojj files are members of my custom mygame.sln. We're reusing
the existing .vcxproj files in the cocos2dx github repo.

Secondly, .vcxproj files should always reference source code relative to
the .vcxproj file. Multiple .sln files can reference the same .vcxproj
file. If the .sln file happens to be in a different network location,
then the .vcxproj source code references will break and create compile
errors because it is referencing source code relative to the .sln file. So
the .vcxproj should not reference source code relative to the .sln file, it
should reference source code relative to the .vcxproj file. The .sln is a
pointer to an array of .vcxproj files and each .vcxproj file is a pointer
to an array of source code files.


What do you think are the downsides of this change?


On Sep 12, 2012, at 9:49 PM, James Chen <notifications@github.com> wrote:

In extensions/proj.win32/libExtensions.vcxproj:

> @@ -51,7 +51,7 @@
>    <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">
>      <ClCompile>
>        <Optimization>Disabled</Optimization>
> -      <AdditionalIncludeDirectories>$(SolutionDir)cocos2dx;$(SolutionDir)cocos2dx\include;$(SolutionDir)cocos2dx\kazmath\include;$(SolutionDir)cocos2dx\platform\win32;$(SolutionDir)cocos2dx\platform\third_party\win32;$(SolutionDir)cocos2dx\platform\third_party\win32\pthread;$(SolutionDir)cocos2dx\platform\third_party\win32\OGLES;..\;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
> +      <AdditionalIncludeDirectories>$(ProjectDir)..\..\cocos2dx;$(ProjectDir)..\..\cocos2dx\include;$(ProjectDir)..\..\cocos2dx\kazmath\include;$(ProjectDir)..\..\cocos2dx\platform\win32;$(ProjectDir)..\..\cocos2dx\platform\third_party\win32;$(ProjectDir)..\..\cocos2dx\platform\third_party\win32\pthread;$(ProjectDir)..\..\cocos2dx\platform\third_party\win32\OGLES;..\;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>

But if game developers update to new version of cocos2d-x, they will
replace all of these projects.
No one will use libExtension.vcxproj without v2.0.* . Do you think so ?
Maybe I am wrong, if so ,please correct me. Thanks.

—
Reply to this email directly or view it on
GitHub<https://github.com/cocos2d/cocos2d-x/pull/1288/files#r1593852>.
It will return false if it fails to create app window or initialize OpenGL.

See Windows implementation of CCEGLView::setFrameSize:
https://github.com/johnangel/cocos2d-x/blob/d071a5e5693dd7c0b517c61f5dea3043ca2c2cb2/cocos2dx/platform/win32/CCEGLView.cpp

More info in my comment:
https://github.com/cocos2d/cocos2d-x/pull/1312#commitcomment-1877279
So did you just add *va_list* to make it compiled ok on <code>Linux 64bit</code> machine?
Yep, i think tolua++ doesn't support *va_lists* .
you should add some phpdoc here
so you're saying just 
````
post.photos ? false : true
````
Nope, on Linux/ARM 4K pages are used.
Maybe a comment explaining why this isn't needed on ARM?
no need to have this on a line by itself
Pull request is now updated with your suggested approach. Worked like a charm.
esta propriedade está estranha (macro portlet no template chart_navigation).. está realmente sendo usada?
Probably we could, but either way I don't think it's not really relevant to this change. I think that exposing `package.json` which is what metadata may be useful, but I don't feel very strong about it. Either way let's have a separate thread for that.
um, is "substitute" a plain string interpolation? Yeah, let's fix that.
I'll extract into a new file dbutils.js.
Great work tracking this down. I'm looking through history, to try to see why it got deleted. Most likely an accident. It's not clear to me how L10n has been working...
I started referencing it in two different places so I thought putting it in config was necessary.
Yeah. I can take it out. I was hoping to be able to bring it back in, but i think I'd have to fork floppyforms, and that's just a bit more work than is worth it for now. I might be able to exempt it and still use it though.... hmmmm.
I hate to be picky, but that is a super-long line.  Any chance you could split it?
Superfluous: that's (trivially) what a constructor does.
But the user can still import interact_sagecell and then mess with it.  Having at least one underscore, and preferably two, is a clear warning to the user that it is considered a private variable.
did you change the whitespace here or something?
Are empty names allowed?
IMO setter name should match with field name
Add description of the meaning of that what is returned.
Why did you remove the above? Is it no longer used?
The abstract Di factory can check if a given instance has any defined instance name. That would disallow instantiating `MyThreateningRMMinusRFFileSystemClass`.
Style police: `pcre *preg`, same for errorptr.
Can you completely remove it?
That's fine. You could target the element using CSS child selectors to enhance performances, but that's fine this way
I'm also not a native speaker :)
Should I also change the original comment a few lines above, it uses the same word?
collections are lazy loading indeed, though once initialized IIRC they currently then read the entire collection, which maybe one day we want to make more flexible.
geokit is the old standard, but I've been moving towards (geocoder)[http://www.rubygeocoder.com/] you might want to look at that.
This is an expected output: in the "basic" functional tests, the default settings are used, so the ``STATIC_PATHS`` setting is set to ``['images']`` whereas the actual static contents are located in the ``pictures`` directory, so it is normal that corresponding static content is not found.
I've even added a specific test assertion to check that accurate logs are generated, see: 86eee303d6ebce3a823d93b99580ba281cd25e79
Note that in the "custom" functional test, the ``STATIC_PATHS`` setting is overridden to ``['pictures']`` so the cross-site links to static contents are correctly resolved.
Should have the Copyright Ghent University line, so we do not need to add it later.
You should add unittests to a different directory in our directory structure, so next to libs and bin. since they are tests...
And that way we can split off the paycheck dependency to only the tests 'package'
Might as well go do 7.1.1.Final here.
fyi, cli has a "version" plugin as well
I didn't use it as it was breaking the unit test (it looks for package.json up the directory tree, but finds the unit test one, not the node-jshint one).
If you drop the --version unit test (which is already tested by cli) then I'd throw in version as the plugin here. (if that makes any sense)
filtering could be configured with the normal createphp type setup and then passed along as config options to the rdfMapper, so transporting this information via createphp wouldn't require any serious code changes. For validation, see my other comment. This needs some sort of support from create.js, since it should normally be handled by the client side, with server validation as fallback only (and of course ideally powered by the same configuration)
Heh--nice catch!
Consistency in how we create class names would be nice.
Hmm. Maybe, but that's a follow-up. We'd have to wrap `render()` in our own shortcut.

```python
render(request, template[, dictionary][, context_instance][, content_type][, status][, current_app]):
    if 'form' in dictionary and dictionary['form'].errors and not status:
        status = 400
    return django_render(request, template, ...)
```
Instead of importing this, you should be using the frozen version of the UserProfile class, `orm.UserProfile`. See http://south.readthedocs.org/en/latest/ormfreezing.html for details.
Ah, I did not notice that.

You should move this onto multiple lines, like so:

setTimeout( function() {
  fetchDuration( delay * 2 );
 }, delay*1000 );
ah I see, you would have to pick the link text that you're clicking on then pass that into the class to form the expected title. that might be a bit overkill.
oops I didn't notice! OK leave it as is :)
Isn't it possible to only use the VFrame API to avoid conversions from ticks to video samples?
The ```-``` refers to the currently authed user. When I go through the example I get data for myself. Is that not the same for you? If so, did you have a fitbit on May 25, 2011 (the date in the example)?
is that necessary? Won't you get a `Event.DoesNotExist` if not found.
i found this error as well and should be fixed in my working copy.
I belive :remote option is not necessary/working on submit_tag as it is specified in the form_tag.
Can you avoid angles within 10 degrees of 90 and 180 to avoid confusion?
We don't use all these spaces any more. Here's the guide for what we want javascript to look like: https://sites.google.com/a/khanacademy.org/forge/for-developers/styleguide/javascript
I don't think it should default to SI - I think that defaulting to the left is fine as long as it's documented.
@XmlIDREF won't work without a String property annotated with @XmlID
This regexp will miss for the following cases:

- uris that are absolute paths without `http://`.  (eg: `url('/path/to/my/logo.png')`).
- server-relative uris (eg: `url(//some.server.com/foo/bar.png)`)
- if the `url` keyword uses different case (eg: `URL(...)`)
- if the url uses a scheme other than `http` or `https` (eg: `url(ftp://foo.bar.com/abc.png`)

The following regexp *should* catch all those: `/url\(['"]?([^\/:'"\)]+(?:\/[^'"\)]*)?)['"]?\)/ig`
(please double-check me though)
Are there concerns with the size of this allocation?
This might be nicer and more versatile to return the sort order that is currently selected and then use assert.equal() in the test itself.
2 blank lines expected here.
Can you also please add the NO_SQL variables to this set.
Please add a line of documentation here as well.
It would be very nice if we could use actors for these events. In the future (no pun intended).
Is this class really necessary? All it does is to fix the text string, so why not instantiate `Hyperlink` with the right parameter when you need it?
Not a big improvement, but sometimes I find it useful to use backticks for method names:

    def `is compatible with 2.9`: Boolean
DokuWiki plug-in config should be used for used settings.
Agreed. I'll amend to take into account single quotes for selectors as well so we are completely inline...
I do not understand this method name.
Don't worry, I absolutely don't think your are too picky, you help me getting better ;) If there's someone to blame it's me for not thinking enough. Keep on !
Согласен, однако это первое решение которое пришло мне в голову. И условие задачи таково что в случайном порядке должен замениться символ на случайный из трех других. Вероятность большого количества выпадения одного и того же индекса как мне кажется не высока. А вообще интересно что лучше отнять еще немного процессорного времени на еще одну итерацию либо немного памяти для хранения промежуточного массива из трех элементов. А вообще конечно спасибо за проверку, подумаю что еще можно сделать )
Ваш алгоритм модификации последовательности не соответствует заданию: в нём сказано, что элементы нужно менять случайным образом (например, если получится, что из последовательности в 100 элементов нужно поменять 3, это будут 1, 56 и 99). Так же сказано, что значение соответствующей ячейки должно **измениться**, если в ячейке было "А" и стало "А" - ячейка не изменилась.
согласен, спасибо, исправлю.
Заказчик дядя Стёпа сказал написать программу, которая из 4 букв сделает последовательность, пусть заполненную случайным образом. Так же сказал, "хочу, чтобы в этой последовательности 30% символов изменилось!". Когда дядя Стёпа увидит результат, и увидит, что вместо 30 символов из 100 изменилось только 20 (недочёт алгоритма), он скажет, что никакой вы не программист, а халтурщик, и хотите его обмануть!
скорее всего эти символы понадобятся ещё в одном или нескольких местах, лучше бы было их определить более глобально - в классе Cell например.
CHUNK_FRAMES seems extremely low. It will a lot of overhead if a DMA has to be fired every 8th sample. Tried something like 32, 64 or 128?
To be clear, shouldn't need step 6.
urlget will raise a PluginError on error, use this instead:

    try:
      res = urlget(q, exception=IOError)
    except IOError:
      continue
Can we line up the asterisks? I noticed it in a few doc blocks.
Still a spacing issue. There should not be spaces between the brackets. And could you also squash your commits into 1? Then I'll merge!
This should have Public visibility for the framework target.
I've always heard (and worked with) 5 cycles. That gets my vote since we're just throwing a warning anyway (and why not warn people more conservatively), but I won't complain about 3.
epochs_ts : instance of nitime.TimeSeries
   The Epochs as nitime TimeSeries object

the reason I am picky on docstring is that they used to format the doc automatically.
I think your tests are keeping the state of the database between test execution.

It doesn't look like each test depends on each other, but it could lead to false positive tests because a previous test state.
Can you replace the literal blocks (::) with specific code-block notation"

```
.. code-block:: bash

some code
```

This makes it very explicit that it's a code block and what language you're expecting the reader to know.
Let's call it `multi_get` rather than `mget`. We've been naming things in the long fashion rather than after their URL segments, e.g. `more_like_this`.
Actually this is just a single class so theoretically it could be a .jar.
this is wrong. It returns an ORM QueryBuilder (building a DQL string), not a DBAL one (meant to build a SQL string)
What do you think about something more like this?

    holds certain dependency injection extension classes, which may import service configuration,
    register compiler passes or more (this directory is not necessary)

There are 2 things I wanted to address. First, the "service container configuration" isn't actually stored here. Second, this directory isn't needed. I think we should include it here since it's generated by the generator bundle so we should explain it, but I actually don't like it because it's unnecessary for most users.

What do you think?
use `.. codeblock:: php` syntax here.
instead of the isdefined formulation for this or time_interp in the code, you can do `start = traits.Float(0.0, usedefault=True, desc='start ???')`
Хм, хотя сработало.
А тут ActionIcon не прокатывает, что ли?
Si estamos haciendo esto para el menú, ¿no habría que asegurarse que funcione para los métodos GET?
Because it uses the PATH constants from `sketchup-stl.rb`.
I would probably split this into two lines
debugger statement.
The original commit of the test has both, I just made them all the same.
So the Jaccard index is based primarily on set overlaps.  If we're treating the input arrays as some kind of count over how often or important a list of items are, where each item corresponds to an index in the array, then from a set point of view, the counts are irrelevant.  All that matters is that the items occurred at all.  In the previous code, we were treating each __count__ as an interesting value to compare, but that really makes no sense.  Under that interpretation if some word occurred 5 times and some __totally **different**__ world also occurred 5 times, then that'd improve the Jaccard Index, which seems to not make sense.
It might make sense to add a link/@see to the design page[1] here and in all SPIs. Would help people understand much better how the whole NBST works.

[1] https://community.jboss.org/wiki/Non-BlockingStateTransferV2
testName missing?
They are two variables with different uses but the same contents.  `j_unique` was in the original code and is used elsewhere in the class; `_client_point_index_map` is new and is only used to convert delaunay's point indices to the client's.  If you prefer I can remove `_client_point_index_map` and add a note to say that `j_unique` has two uses.
I don't know if this ever made it into the code guidelines, but we've tried not to have unicode-encoded files like this, but instead use `\u` escape sequences for unicode characters.  I know it's not as obvious -- but some developers are still using editors that don't handle unicode encodings correctly -- I'm not one of them ;)
We should probably just delete this line
Inconsistent indentation.  Personally i prefer 'foo="BAR"' (no spaces) for function calls and PEP-8 agrees: http://www.python.org/dev/peps/pep-0008
Also I think this locator is an exception and can be inline rather than a variable. It's not related to the application under test and wouldn't go into a app object.
Why did you drop the TODO?
I think the same. A space after the cast is more readable.
The tree explanation should go at the top.
@sritchie Yo Dude, I'm seeing some strange `Unable to resolve symbol: integer` behavior with 1.4:

```bash
eightysteele $ lein new app
Generating a project called app based on the 'default' template.
To see other templates (app, lein plugin, etc), try `lein help new`.
eightysteele:~$ cd app/
eightysteele:app$ cat project.clj 
(defproject app "0.1.0-SNAPSHOT"
  :description "FIXME: write description"
  :url "http://example.com/FIXME"
  :license {:name "Eclipse Public License"
            :url "http://www.eclipse.org/legal/epl-v10.html"}
  :dependencies [[org.clojure/clojure "1.4.0"]])
eightysteele:app$ lein repl
nREPL server started on port 55364
REPL-y 0.1.0-beta10
Clojure 1.4.0
    Exit: Control+D or (exit) or (quit)
Commands: (user/help)
    Docs: (doc function-name-here)
          (find-doc "part-of-name-here")
  Source: (source function-name-here)
          (user/sourcery function-name-here)
 Javadoc: (javadoc java-object-or-class-here)
Examples from clojuredocs.org: [clojuredocs or cdoc]
          (user/clojuredocs name-here)
          (user/clojuredocs "ns-here" "name-here")
user=> (integer 1)
CompilerException java.lang.RuntimeException: Unable to resolve symbol: integer in this context, compiling:(NO_SOURCE_PATH:1)
```

Are you seeing this on your side?
@bp323 also remove the space before `:`
FYI - I had to use `!important` in here because, as soon as the carousel gets initialized, it is set to 0 with inline CSS
Added comment - was a general bug and not sidebyside related. Effected page/settings/visibility. Maybe take a look as I think it's unusual that a fieldgroup is inside another fieldgroup.
Maybe we should only have users choose the option to include custom form fields and report personal info - but have all the rest available? 

Yeah, worth having a conversation on for the next iteration :).
This should probably be `||` instead of `&&` as I believe specifying just one or the other is also technically valid.
I think you asked me whether this was ok during the sprint.  I don't remember what I said but I don't think that it is :-)

This seems too much like a side effect (and too brittle when we think about future code that might use this partial).

Can we instead just pass in a parameter like "make_search_remote"?  (or something like this)
I don't understand this change
WSF_FILTER_CONTEXT_HANDLER inherits from WSF_CONTEXT_HANDLER

so it is enough to put WSF_CONTEXT_HANDLER

The advantage is that you can end the filter chain with a pure context handler
but still you can have a chain of many filter objects

I guess I am missing an obvious use case, in this case, could you tell me which one?
diese Berechnung gehört für mich nicht in diese Klasse --> High Cohesion
Vielleicht in den HMACHelper
Was meinst du mit Guard? Bin auch der Meinung, dass man das irgendwie auslagern sollte.
I store "true" or "false", then this is better than check for string (although it may be more logic to store 0 or 1 and use +val, need to change that rly)
yeah i can move it to a function that returns a boolean. we could move req.session.nickname[channel] to req.session.nickname.channel but that would assume nickname will have other attributes? or is it too much overhead to add another layer? i'm fine w/ either option
I wrapped the object in an anonymous function, and set it as an object on the page at the bottom.
Wait, it **removes** the float? Weird.

Maybe `makefile` should use `node_modules/.bin/lessc` so we all use the same version?
yes, wanted to make it less noisy as the error/warnings are expected
have pushed a fix that silences only that particular warning.
Should be "descargas" instead of "modificaciones"
As discussed, assignment might be nil, so the following line may throw an error.
No no. Not this one. This one is OK to be named `access_token`. The one which is passed to the `authenticate` method.
Aaagh. Why you simply don't use jQuery to attach directly to DOM? Why play so much with strings? You would not even need temporary array.
What was wrong?

Yes, prease cleanup this code. Also rename argument to `twitter_token` and for Facebook to `facebook_token`.

Yes, do assert similar to how we have it for registration.
I would store this before those `if` sentences.
*old* -> *current*
Space before `(notifications)`.
Maybe the problem you are having here is that you are overriding the whole `js` block and not just adding to it. ;-)
So I should define a datetime field, which would get a value of current time when information would be fetched? But all stations wouldn't have the same value for this field, because there would be differences in miliseconds?
And how would the query in get_stations_nearby look like? We have to order twice, once for distance (to preserve order in distance from node) and once for this new field. We could first order by new field, retrieve just first 32 stations, and then order these by distance. But as some stations would be updated a few times in some period, and others never, we can't just take first 32 stations. I still can't see how the query could be simplified with this field.
Delete it. `git rm`.
I know you're just porting this, but VERIFY_NONE is bad.
We could also put this code in some .cpp or .h file and include it into multiple projects. Since the git library is statically linked and only used objects are included in the binary both should be ok.
No need, a git rebase -i would be fine
Using smart script you can cast spell like you done it in code: target->Cast(target...
m_lastDamagedTime -> _lastDamagedTime
That's cool. Have a look at my partial branch.

Pete
...from my mobile.
On Sep 17, 2012 9:36 PM, "Pawel Kozlowski" <notifications@github.com> wrote:

> In grunt.js:
>
> > @@ -53,7 +53,7 @@ module.exports = function (grunt) {
> >        }
> >      },
> >      watch:{
> > -      files:['<config:lint.files>', '<config.src.html>'],
> > +      files:['<config:src.js>', '<config:test.js>', '<config:src.less>', 'src/modules/*/partials/**/*.tpl.html', 'src/index.html'], //need to have a path to index.html, otherwise watch won't pick it up
>
> sorry @petebacondarwin <https://github.com/petebacondarwin> I had grunt
> watch crashing on me with your changes. On top of this it:
>
>    - wasn't watching partials
>    - wasn't watching less files
>    - was watching grunt.js, don't know if this can even work (I'm not
>    sure that Grunt will pick up changes to its config files) I know that the
>    above change is not DRY enough but I'm sure that we will be going over
>    grunt.js many, many times...
>
>  —
> Reply to this email directly or view it on GitHub<https://github.com/angular-app/angular-app/pull/5/files#r1622798>.
>
>
Skip the entire comment and use `{@inheritDoc}` (less work later if things change)
Yes, I know I am a PITA:

```php
if  (!include_once __DIR__ . '/../../../autoload.php' && !include_once __DIR__ . '/../vendor/autoload.php') {
```
Not sure it should use an `authenticateCreateQuery()`. A query is very DB-specific, while the `ObjectManager` doesn't assume there's a DB. Will think about it, doesn't need to happen now anyway.
This gitrepo doesn't seem to exist
Since `heater` is a class, you should case it as such: `Heater`
maybe we can have some logic with a list and use getattr(args, value) instead if listing all of this explicitely?
What do you think about not disposing this?  It is injected into this instance (this instance did not create it) and there it doesn't own it.  Seems like it is whoever created it's responsibility to dispose of it (if necessary)...
> You are right, mostly it doesn't. Planet with albedo = 0 should be basically iceball. It can happen if the planet had thick atmosphere which froze, but planets with thin atmosphere should have higher albedo.

Albedo is one of those quantities:) It's been defined by how much it reflects rather than absorbs. So an albedo of zero is a black body radiator. Some of the newly discovered extra-solar gas giants have an albedo close to zero so, it's probably reasonable.

Also http://en.wikipedia.org/wiki/Albedo#Astronomical_albedo says that asteroids can be very black.
I agree, naming here is not good yet, especially for the argument. In general I like to keep the distinction _idx for integer and _mask for boolean in the name.

I changed at some point whether iin are inliers or outliers. (So far I only used support in the case of distributions.)
Ah, good. I'll have to catch up (currently homeless with limited internet). Looking into the changes needed for FFT.
This should probably be done by someone who knows what state these functions are in. I removed some warnings here: 76578bf

Let me know if some of them should stay and/or which others should go.
I'm leaning away from using branches for anything other than nightly, since a branch's endpoint changes with time - if we were to check this specific code out again in a year and try to build the same 1.0.2 package, we would get something different (assuming new stuff was commited to the branch). Using a tag or sha means the specific version of the package is not time-dependant.
Import only modules, not directly its content.

This is wrong. Or `from piplmesh.account import models` or use [relative imports](http://www.python.org/dev/peps/pep-0328/#rationale-for-relative-imports).
It's easier to see which elements use this property since there will be quite some after we are done with it.
This might be more maintainable as `self.class.concrete? @type_name`
OK, convinced by the second bit.  How the first bit works, I confess, still feels a little strange to me, but that is not the fault of config!

It may be Numerical Recipes-indoctrinated superstition, but I have a slightly hard time wrapping my head around the fact that it's OK to simply seed a random number generator with 50 successive integers (say) and have the outputs be truly random with respect to one another - I'd always tried to initialise RNGs once and then let them update the seed as necessary.  

And I guess I'd thought that our multiprocessing would work a similar way: that we'd start off with `nproc` separate seeds, and then each postage stamp would somehow pass its updated seed onto the next.  But people who know more about it than I have mentioned on other threads that successive integers should be fine... It just feels wrong :)
Needs a test as well.

Example of deprecration warning, @jrus:  https://github.com/jrus/balanced-python/blob/aa3399afa252ede433e0cbaaf4c02d9adc9afafc/balanced/resources.py#L696
they're slightly different, but i've created an Identifiers and cleaned it up a bit more
entity is included in this.toString so removed here
s/sel/selenium

also on lines 226 and 229.
Why did you commit this but comment it all out?
there's a pattern you can reuse from some other code..

import org.jclouds.util.InetAddresses2.IsPrivateIPAddress;


      builder.publicAddresses(Iterables.filter(addresses, Predicates.not(IsPrivateIPAddress.INSTANCE)));
      builder.privateAddresses(Iterables.filter(addresses, IsPrivateIPAddress.INSTANCE));
Perfecto, dejo el archivo tal cual estaba y vamos avanzando hasta la configuración en sql.
I belive your suggestion comes from the philosophy that this should be
a "true thin client" that is always able to use ActiveResource in a
vanilla sort of way (which does sound nice).

Then, in the ideal world, I believe we would want something like:

    pt = AeolusCli::Model::ProviderType.find(:name => options[:provider_type])
    p = AeolusCli::Model::Provider.new(:name => provider_name,
         :url => options[:deltacloud_url],
         :provider_type => pt,
         :deltacloud_provider => options[:deltacloud_provider])

The immediate problem is that the api does not support the above
ProviderType.find [1].  The code above generates a request like

    GET /api/provider_types/%7B:name=%3E%22mock%22%7D.xml

which returns a 404 response of

    ...The action 'show' could not be found for ProviderTypesController

BEGIN WORKAROUND THAT DOES NOT WORK

I've tried instantiating a ProviderType locally with new()

    pt = AeolusCli::Model::ProviderType.new(:name => options[:provider_type])

instead of using ProviderType.find().  However, the xml request generated by

    p = AeolusCli::Model::Provider.new(:name => provider_name,
         :url => options[:deltacloud_url],
         :provider_type => pt,
         :deltacloud_provider => options[:deltacloud_provider])
is

    <?xml version="1.0" encoding="UTF-8"?>\n<provider>\n  <deltacloud-provider nil="true"/>\n  <name>mock-test-provider</name>\n  <provider-type>\n    <name>mock</name>\n  </provider-type>\n  <url>http://qeblade30.rhq.lab.eng.bos.redhat.com</url>\n</provider>\n
and the error from the server is

    <message>ProviderType(#59825120) expected, got ActiveSupport::HashWithIndifferentAccess(#24486100)</message>

END WORKAROUND THAT DOES NOT WORK

Anyway, this is why I'm currently building a hash of
provider_type_name => provider_type_object and why I added in the TODO
of making sure the provider_type name exists.

[1] https://redmine.aeolusproject.org/redmine/projects/aeolus/wiki/Provider_Types
In this case javadoc become useless, when you write HttpRestProvider implements HttpProvider. It's implicit.
Just a quick check: where does this end up being raised to?
Выноси в функцию, рекомендуется делать так, чтобы читалось :-)
Много пробелов
Смотри комментарий к методу readLine - он может вернуть null.
a statistic is not thread-safe, so this introduces a race condition

I am going to rework the whole Statistic design (it's a prototype mess anyway :) to fix this.
Didn't throughly test this url mapping, I think it probably should of been a conditional param.
@shacker can you change this to "Canvas Assignments" so it's more inline with "Google Tasks" ?
Now, now boys. Keep up the questions, and let's try to get to the interesting stuff.
Why are these changes not also in the changeset for `php54`?
I would really like to fix this if possible. I'll come hassle you.
This is only for readability purposes, because it overwrites the selenium click()
same here^
The element here already has a unique identifier, so I'm not clear why we need to locate based on the contents of the data-context attribute.
プロパティ形式でアクセすることができる
↓
プロパティ形式でアクセスすることができる

typo
You mean the liveshift class? I wrote it. I used the same copyright helper as all the classes.
Why do you prefer `null` over `Option`?
you are right, I have to redo this.
If this is an unsupervised method, maybe it should be in the unsupervised part of the documentation, with cross-references.

I am just wondering what the best way to make this discoverable is.
You should explain what passive aggressive learning rate mean in practice here and also give the mathematical formulation of those losses in:

http://scikit-learn.org/dev/modules/sgd.html#id1
What do you suggest instead? We do that as well in forests and trees. Hence I was thinking it was not problematic. (And I don't see why it is?)
по идее здесь должен быть обычный find

    искать потом можно так
    find({categories:ObjectId(1)}) или find({categories:{$in:[ObjectId(1), ObjectId(2)]}})
Вон у Йосси Крейнина почти на эту тему хороший пост – http://www.yosefk.com/blog/api-users-api-wrappers.html , но это так, если интересно
Mind declaring an array and using forEach on the entries? Would be also nice to have those sorted alphabetically.

The application id we have to check for the specific entry: '{ec8030f7-c20a-464f-9b0e-13a3a9e97384}'
It's customary for all the ImageBufAlgo functions to return a bool indicating that it worked ok, or had an error.
You don't need the first fork.  Essentially you redirect the output of the current process into the grep process, and then yield in the current process.
I don't think this is the place to do that.  This script is for project creation, so it get run only once.  We'll need to copy or at least package the resources every time we build.  So this step during project creation is not needed and can be removed.
Une fois Apache redémarré, activer le mode pseudo-streaming h264 dans le setup de l'installation.
Yep, it'll be slower. My Ruby-fu isn't strong enough to reduce the list correctly, but `Rummageable.index` expects a list of documents + an index, so you'd have to reduce it to a pair of `[documents], index` to do it quicker.
Extract this to a separate function? I think we should also print a warning, like "This feature will move when Buster.JS 1.0 is out. It will be part of the (as yet non-existent) buster-ci tool."
@rhauch I'm not familiar with the regex libraries, I guess they are a collection of analyzers?
I have no problem in adding some more analyzers which are of common usage, just let us know exactly which jars you need.

what is the problem with the slot name? can you not use it as "main" hibernate search module?
basically, why are we dealing with volatiles in this case, but doing as I would expect on 7e087e5a6aff4c42c796781ef7c6b96922b46fa1 ?
This is named from the parameter on the function you're calling.  In this function it should really be fieldContainsEscape.
Is there any reason we are not using standard SOAP calls?
A similaI call to DFUInfo (for example) would look like this:

```java
		WsDfuServiceSoap service = platform.getWsDfuService();
		if (service != null) {
			DFUInfoRequest request = new DFUInfoRequest();
			request.setName(info.getName());
			try {
				DFUInfoResponse respsone = service.DFUInfo(request);
//snipped
			} catch (ArrayOfEspException e) {
//snipped
			} catch (RemoteException e) {
//snipped
			}
		}
//snipped
```

Gordon.
The main reason for this change is to pause the DFU execution queue, so ops can work around without stopping the queue and killing jobs.

If part of the jobs comes from the monitor queue, pausing just the dfuserver_queue won't accomplish that.
This is an interface, so public is the default.
No, it's just the way diff displays things - not an indication of a problem.
cast should be to memsize_t not size32_t
Yes. Some files fail with, while others fail without... Maybe the flags should be more specific to groups of files... Not sure how to do that, though, with the current approach, without bloating the test scripts...
These should be offset_t as well, file might be bigger than 32bit.
This is still confusing c versus c++ compiler. Checking the version of the c++ compiler before deciding what options to pass to the C compiler seems wrong.
Can we remove these `[CDATA` junk things please?
We're using HTML5 anyway.
Is pep8 bitching about trailing whitespace now?!

I have a hook in my editor that rstrips all whitespace when I save so I've never experienced this. 

Check that your pre-commit hook actually runs the same version as jenkins does.
I disagree because this is not an exceptional condition - we know this will happen. Exceptions are not flow control :)

They are more expensive for one thing.
I'm pretty sure this is wrong. As an example:

Suppose your subnet is `192.168.0.0/25`. This means your IPs will be either `192.168.0.*` or `192.168.1.*`. This means only two quads can actually be used in the match, but `32/$mask` will return 1 in this case, meaning we will only match against `192.168.0.*`.
This won't parse on 1.8.7 (which we still support, right?).   Seems funny to use both hash syntaxes on the same line, anyway.
I don't know... I think it is safe to remove the `ctor` from the class. And by not adding it will make the process of creating new objects a bit faster.

Regarding your previous question, I'm not sure which pull request should I fix. The pull request that I sent to cocos2d-html5-tests ? Thanks.
should probably have the request endpoint match, and somehow not to folks not to change the callback endpoint.
throw "Exception(curl_error($curlHandler), curl_errno($curlHandler));" here
when you make first a boolean

if first
Keep `rotate` in _warps.py or move it to _geometric.py?
Would you mind adding a commit for that?
ResizeInstanceVolume sounds nicer. NGTHTPR
Killer name.  :)
should be "group of menu items identified by a"
No doctype here?
I want to make sure we're both on the same page. Did you remove this section because it's covered on Forge? What if someone is just looking at the source code? I suppose they'll have the metadata..
`desde tu` ;)
No, just copy, no paste. ;-) That's left over from its origin and was just never touched. I guess it's best to completely delete the README as it doesn't make any sense when the template is not downloadable on its own.
вроде ни строчки по-русски в геме нет.
либо добавить везде, либо убрать везде.
Documentation here clearly defines what the requirements are, so the note is redundant.

LGTM.
> I'm unsure which destructor you are talking about? The temporary t ? If you know how to avoid its call, I'm all ears, but I don't think I have a choice...

How to avoid? Something like this:
```D
initialize(result);
foreach(i, ref field; result.tupleof[0 .. args.length)
    field = args[i];
```

> Other than to emplace each argument individually element using ParameterTypeTuple I dodn't see how to avoid the call.

> I actually did that in one of my versions, but the orverall code was too complicated, IMO.

Sorry. So you don't need my "how to avoid". But is it so complicated?
Maybe rename "listener" to "game". Because we're not using it as a simple listener anymore (to throw events to) but also to get info (e.g. isPaused).
The name is currently confusing.
Didn't see the problem. Maybe use a RegEx?
How about `@TheOne` :)
@stof do you happen to have an example?
On desktop (and in your previous string), these are ordered `Bookmarks, Passwords, History Tabs`.  Be consistent.

These will need to be resources (possibly an array) so that they can be translated.
COOK-1733 created for that task.
The feature didn't work anymore, and since we are planning to rewrite it anyway to work with side-by-side I thought we'd place it in the backlog. Let me know if we need to restore it now.
yes as the AbstractBuild.getRootBuild() was added Jenkins 1.421 
http://javadoc.jenkins-ci.org/hudson/model/AbstractBuild.html#getRootBuild()
Order matters here ... pkttype I think belongs between :port and :name.
A unit tets to check this would be handy, though :-)
hmm... nope.  just an oversight on my part.
I'm confused about what we're trying to achieve here. It sounds like we're trying to standardize some basic naming conventions so that functions and classes from other projects don't feel foreign when imported into another project. But, if we standardize function and variable names to use camelCase instead of underscores, won't that make PHP's own standard library feel foreign? I don't see how we can possibly standardize on camelCase functions names without undermining the very purpose of this standard.
Does this need to be a function if it isn't doing anything?
Can we remove the `em`?

And maybe only use margin-top? -- Not sure about this though, since it's possible that we want to override the right/bottom/left values.
You shouldn't inclue a private key file in the repo!
Sure, but how far are we willing to take it (e.g., can not contain 'LIKE' 'AND' etc...)? May be simply removing the 'pattern value="([a-zA-Z0-9])*"' is all that's needed, leaving out as String type as it was before. wdyt?
How about to move these new lines before one about setting 'message-converter', to make the code more more logical and consistent?
Yes, it is. But it is just for each method. So, incapsulate `consumer` variable inside the each method.
Also why do not use here
```
@RunWith(SpringJUnit4ClassRunner.class)
@ContextConfiguration
```
And @ Autowire your consumers
Given these else conditions are identical, I think you can significantly reduce the complexity by using

    if (this.parsePayload && payload instanceof ...) {
    ...
    }
    else {
    ...
    }
I'v added <i>this.collection = collection == null?DEFAULT_COLLECTION_NAME:collection;</i> instead.
what about 4th? May be I am missing something but you mentioned that the attempts went up to 4
What if there's an escaped '=' or one inside quotes?
There's no need to look up the Drupal username because it is not necessarily what's needed to authenticate against LDAP.
here sslChecks is passed in - should it not be sufficient to check that for true/false instead of always having to do the useragent checks/munging blindly ?
@gordonslondon you also need to alter the ``page__block_audit`` table ;)
eecution => execution
OK, so that's where they are now.

If we're going to do this I'm happy to do a rename to move the files into Code52.i18n.MVC project and simplify the solution.
This is the second thing I changed, the rest was whitespace due to autoformatting
If there was an error, we should just surface that error and not alias it with our InvalidResponseSyntax error.
@christianv I get a missing template error on this link.
Why are you testing with ``method_exists`` ? you should check against the interface ....
Constructor doc should always start with "Initializes a new instance" and it is not necessary to mention the type name.  So here, you'd write, "Initializes a new instance with specific radii."  This is really nitpicky, but consistency is important, especially in documentation.
Here missing php doc block comment.

You can explain what this class does
Should fix up the test message to be more meaningful.
No preference either, as long as hub_cap is fine with it :)

On 7/5/12 11:33 AM, "Michael Basnight" <mbasnigh@rackspace.com> wrote:

>doesnt bother me either way :)
>________________________________________
>From: Mike Asthalter
>Sent: Thursday, July 05, 2012 11:30 AM
>To: Nirmal Ranganathan
>Cc: Cloud DB
>Subject: Re: [reddwarf_lite-integration] final doc updates for 7/2/12 EAP
>release (#103)
>
>Hi all,
>
>Here is what is used in the Beta version of the Cloud Identity Client
>Developer Guide:
>
><?xml version="1.0" encoding="UTF-8"?>
><auth>
>  <apiKeyCredentials
>    xmlns="http://docs.rackspace.com/identity/api/ext/RAX-KSKEY/v1.0"
>    username="jsmith"
>    apiKey="aaaaa-bbbbb-ccccc-12345678"/>
>  </auth>
>
>I would be happy to change to this for the next version of the doc if you
>all prefer.
>
>Please advise,
>
>Mike
>
>
>
>
>On 7/5/12 11:23 AM, "Nirmal Ranganathan"
><reply+p-1106263-90f89f6d31b89094c3c0826baeae7fbf1114bcc6-1015169@reply.gi
>t
>hub.com> wrote:
>
>>> @@ -0,0 +1,14 @@
>>> +POST /v2.0/tokens HTTP/1.1
>>> +User-Agent: curl/7.21.0 (x86_64-pc-linux-gnu) libcurl/7.21.0
>>>OpenSSL/0.9.8o zlib/1.2.3.4 libidn/1.15 libssh2/1.2.6
>>> +Host: identity.api.rackspacecloud.com
>>> +Accept: application/xml
>>> +Content-Type: application/xml
>>> +Content-Length: 88
>>> +
>>> +<?xml version="1.0" encoding="UTF-8"?>
>>> +<auth>
>>> +   <apiKeyCredentials
>>> +
>>>xmlns="http://docs.rackspace.com/identity/api/ext/RAX-KSKEY/v1.0"
>>> +       username="hub_cap"
>>
>>yeah that's just a dummy api key. But you are right we should probably
>>change that .
>>
>>---
>>Reply to this email directly or view it on GitHub:
>>https://github.com/hub-cap/reddwarf_lite-integration/pull/103/files#r1106
>>2
>>63
>
Note this line is unnecessary - as line 17 imports the module
You could use: ``if 'date_from' in request.GET or 'date_to' in request.GET``

We should allow only one of them to be set.
> In order to use constraint annotations for method or constructor parameters, their element type...

I find this ambiguous. what's about:

In order  to use constraint annotations for method or constructor parameters, the constraint annotation itself must be annotated with @Target(({ METHOD, FIELD, CONSTRUCTOR, PARAMETER }))

Maybe also a link to section "3.1  Constraint annotation"
Sounds fun. I like it more than "multi_handle" :)
Change the name becouse of API of google calendar. I change but i put an alias. What do you think ?
Thats lovely, handled (I can't bring myself to say "fixed") in 8fb4175b9fd6e50817e6acc3713245384ab6da18
With the exception of BinaryToPut and PutToBinary I would think that someone has written those functions before. Do we want to reinvent the wheel here?
why commented ?
The old suggestions for ZF1 were like 

`locale(s)/en_US/messages.mo`
"If the URI scheme portion of the passed path is not a member of"
Oh right, forgot about that. I was testing a theory that turned out to be true...just forgot to clean up afterwards.
useful descriptions pls
das 'pass' kannst du dir sparen
For robustness, might want to check pos >= 0 - or at least validate what will happen if an invalid query string is passed in. Might be worth using a more robust query string parsing library - we have some in ASP.NET and other parts of C# / libraries that we could consider using.
use page_url
Please revert it
Здесь проверятеся несоответствие спелла текущему окружению. Т.е. твой баг надо искать тут.
Lint complained when I did that...I believe it said "Bad constructor"?
Typos: serach -> search, key word -> keyword
Why was the -lang-c++ flag added?
Don't ever trust user input.
If the user added a @ in front of their Twitter username, we will remove it, and then put it back.
If the user neglected to add a @ in front of their Twitter username, well, it's there now.
But on the other hand it will be confusing to D programmers. Because `X86` is not defined on `X86_64`, people write code like this:

    version (X86)
    {
        32-bit x86 code ...
    }
    else version (X86_64)
    {
        64-bit x86 code ...
    }

The equivalent code for MIPS would then be wrong.

I get your point about boilerplate but consistency is more important IMHO.

As an alternative, we could have `MIPS`, `MIPS32`, and `MIPS64`. That approach still suffers from the consistency issue but not as much.