labofsahil/github-event-dataset-2012 huggingface dataset:
Please use `data` argument to `getJSON` and don't try to construct query string manually:

> Data that is sent to the server is appended to the URL as a query string. If the value of the data parameter is an object (map), it is converted to a string and url-encoded before it is appended to the URL.
Remove this. :-)
so u want me to save all data like windDirection windSpeed humidity pressure cloudiness fog  lowClouds mediumClouds highClouds at all possible timespans ?
but this data is not accessible at all hours ?

so i should make an array for each data and save it ?
Ok, I see that the requirement for ASCII with = excluded comes from Vorbis Comments, but what about ~?
Поставить до валидации и проверить что если ошибка дальше ничего не диспечерзуется
And this is why you don't comment on pull requests when you're on cold meds. Right.

However, I think we can probably make adjustments to this to make it work better with Mongo. We wouldn't want to pull all results into a list unless we explicitly limit the size of the results somehow. I'll have to give some thought as to what's the most efficient way, but as a start we can do `skip`/`limit` and `count` -- though this becomes inefficient for large collections.
remove line 8625
what about objects without a ``__toString`` method ? I see a common case: ``DateTime``.

And btw, the common way to use it is to cast as string, not to call the method directly (some classes implemented in C can be casted as string without defining this method IIRC)
I have a comment on this
Oh sorry @koblas, I misinterpreted. Yes, you're right, I don't know what I was thinking when I wrote that. Like I said, I ended up scrapping that line anyway :smile:
Well come up with a name that is lower in the alphabet and will use that ;)
typo spotted, fixed in c889033b044ac6d0eec09a24447b51e4f07852d8
Remove Temporary hack.
No, that's wrong. There are a few more that can be private now. Have changed this now, `_current_request`, `_current_response`, `_set_is_finished_on_responsefinished`and `_called_get_body` can be private too. Also some properties were missing there and were just set later, I hope I found them all now.
Good point. I was overly cautious here.
Why the `else`
Good, fixing.
Nitpicky, but the parens around `token[STR]` are unnecessary.
Also these, shouldn't these be in PSOL?
Maybe cache the results of Name() and Value()? I don't think gcc can determine the pureness of those methods unless you turn on LTO. (You can always look at the generated assembly, of course.)
I have been testing this fork and have not seen this error. Can you describe the steps reproduce?
for score we should also have a scoreobs.
Not used yet, but available for specification tests and for sandwich covariance matrices (not yet included from GenericLikelihoodModel)
@xsuchy We could, but we can't use either of the two entries that were in it - the foreman user, or the path to foreman since systemd doesn't interpolate variables in either the user field or the path to the binary.  We could add it, but it would be empty - I can't see an easy way to provide the same functionality as before and think it has very little value, given the RPM is based around that user/path.
Don't we need this template anymore?
Actually an empty `alt` attribute is valid markup and can be semantically correct - for example, an image which is already described in text via a caption beneath, or an image that conveys no semantic meaning (such as page furniture or a tracking sprite).  An _absent_ alt is invalid.
This is arabic.  For those of you who are curious, here's a translation:

The message you want to convey to you ..
The rockets of the resistance has reached Tel Aviv and Jerusalem occupied West Bank and to all settlement blocs near the Gaza Strip .. The Taúratkm flying in the skies of the Gaza Strip will not fly anymore. And Oltm that move along the line Heh came under Mujahideen strikes and warlike Svenkm off the shores of Gaza is under Targeting
You now choose between two things only two
(Either leave Palestine, or that Tmoto at the hands of the resistance)
We will provide you our services fast Barsaleckm to die slow ..
This and we wish you fire Mnamin where, God willing,

...
We've had this commented out all this time. What does this function do? Maybe a docstring would be helpful?

I understand that an id is being set, but what is this used for?
Hmmm, I think you might be right on all counts - certainly it is difficult to argue that the current half_light_radius inheritance is aiding clarity, at all, and it's not saving much boilerplate in those cases where the default definition can be used.  I would be happy to make the changes you suggest and strip this from the RadialProfile.  As you mention, this makes the RadialProfile pretty empty... (I'll comment more below)
Thanks . Doing it right now .
You can also use `EntityTestUtils.assertAttributeEqualsEventually(entity, Startable.SERVICE_UP, true);`.
I don't think the DTO class is the right place for this method. It's going far beyond what a DTO normally does (i.e. used for transfering data). I think it would be better to move the business logic / load etc somewhere else (but not sure where yet).
Not sure about this; I'd class Error as normally fatal as well (e.g. OutOfMemoryError etc then all bets are off).
_so_ inutile
`img` -> `images`, for consistency with all other demos (and template).
They both use milliseconds since the epoch, so the tests pass. See the comment below though, can probably scrap and just use the date constructor.
I wonder if we can somehow share the add/remove/move logic between Playlist and ListeningRoom. I added the m_savedInsertPos + move/etc logic to handle spotify syncing (we need incremental changes not just full revisions),  but it seems fragile now to have to maintain this tricky code in two places.
will it be no better to move file path code to common location. The code is also present in pzp/lib/session_common.
On 06/03/2012 23:21, Fabien Cazenave [:kazé] wrote:
>> +
>> +  function newListItem(ssid, network) { // .ssid, .bssid, .flags, .signal
>> +    var li = document.createElement('li');
>> +    var br = document.createElement('br');
>> +    var span = document.createElement('span');
>> +    var small = document.createElement('small');
>> +    var wifi = document.createElement('span');
>> +    var signal = document.createElement('span');
>> +    var secure = document.createElement('span');
>> +
>> +    // ssid
>> +    span.textContent = ssid;
>> +    small.textContent = (network.flags) ? network.flags : 'open';
>> +
>> +    // signal
>> +    var level = Math.floor(5 * (network.signal - 101) / maxSignal);
> because I want the end result to be between 0 and 4 (inclusive).
> I could use '100' with 'ceiling' instead if you prefer.

Sounds more understantable to me.
i thought nanoregen aug is too cool, healing all the kinds of damage simultaneously, i tried to make it do a list of injuries, pick one and heal it a bit and heal another one to sum of same 3 hits.
TAG is never used.
Is my patch not working, or throwing errors for you folks?
We don't actually support Weld on EWS, so we need to explicitly call out in a comment that this is an unsupported component.
I think this might be a global leak?
What is this meant to be doing?
Nice job in adding these, I would have totally forgotten!
Fixed post merge: https://github.com/opscode-cookbooks/apache2/commit/a920e047b230e289ea8ee5bd807b71ae2b74106c
I'm actually running OSX 10.6 64-bit, python 2.6, numpy 1.8.0dev. Maybe it's a 32-bit/64-bit compatibility issue? It could also be an issue with different numpy versions. I can test the numpy version.

(Note that the issue in the Travis build is a different error and can be fixed with a change in the test tolerance.)
I think the comment got lost during the last commits (damn you github): shouldn't you sample here?
Actually no, it is required since n_node_samples and n_total_samples are both ints, and that int / int is an int.
I would explicit the argument name for "diag"
check that X is coo
Good question. I didn't do anything, this code just isn't used anymore, and it turns out that I can comment it out, together with `predict`, `predict_values` and `cross_validation`.
Yes I agree..
use `length`
在这里添加了一个注释。
What comment is that?
The docs said: "One stop shopping for device battery information.", so seems useless to have it as a member.
this needs to be TiProxy::initializeTiObject.  The TiUIBase one will create a Ti.UI.View of it's own which is not what we want.

You could had a `Q_ASSERT(getObjectType == N_TYPE_VIEW)` inside TiUIBase::initializeTiObject, to guard against doing that by mistake.
Un business field n'est recherchable que par un utilisateur ayant accès à celui ci
Un tableau des types mime
Yeah, it appears to be missing from OpenIndiana (successor to OpenSolaris), but since I couldn't get the ./configure script to get even that far, I couldn't test further. If necessary (and someone can help with my build issues), I'll write an implementation of fgetln too, which should get things working on even the stuffiest of build environments.
`Coordiantes` > `Coordinates`
I'm liking this annotation, but one thing we should do is to make as many tests as possible in the hibernate-search-engine module (vs. the hibernate-search-orm).

Which doesn't have this dependency. Ideas?
(Not something to be solved in the scope of this pull)
I would expect
expandExprText(body);
appendAnnotation(expr);

My preferred syntax of the day :-) for annotations would be

%n = %1{symbol x}

i.e. each annotation in curly brackets.
No, it's deliberately allocating until it runs out. (this is the unit test)
Not convinced this is a good change.  That comment is significant for the version that batches up adds to reduce the lock contention.
If that is an independent class then there is no problem, and probably a good change.
Pedantically, yes. For now, it'll just ignore.

Doing so, might break ECL code that rely on the generality of file handling, so I don't think this should be done lightly.
I should probably replace the settings bit with the operating_systems returned from the model call. correct?
i just filed the style that was already there, not sure why it's like that...

I am going to file a bug to excise currentversions from here (it shouldn't be exposed to templates, releases is a better data structure), let's handle it there.
We desire that look of this text. Will changing it to `<code>` wrap nicely and still look great?
Remove this line and dedent. Move it to line 329...
Ah.. I'll fix
On Aug 30, 2012 8:57 AM, "Bryan McLellan" <notifications@github.com> wrote:

> In test/kitchen/cookbooks/bluepill_test/attributes/default.rb:
>
> > @@ -0,0 +1,5 @@
> > +require 'securerandom'
> > +
> > +default['bluepill_test']['service_name'] = "test_app"
> > +default['bluepill_test']['tcp_server_listen_port'] = 1234
> > +default['bluepill_test']['secret'] = SecureRandom.uuid
>
> This test won't run on 1.8.7 :(
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/opscode-cookbooks/bluepill/pull/7/files#r1490724>.
>
>
Тем более непонятно к чему у тебя здесь символ :dog2:
You're quite right - I can only imagine that this was an artefact of an earlier attempt before I found `src/capybara.js`. I'd be happy to fix in a subsequent commit, or to amend this commit. Do you have a preference?
more in room discussion, but deciding to punt on the new categories for now. I'm trying to avoid the timesink of dates and date manipulation and getting the initial category implementation up
I relocated the "access_granted?" method because each proxy has its own requirements.
I think this would be easier to maintain if it were refactored to call a series of methods that were like `_findHighscoreMatchesByWeek(matches, week)`. Iterating over the data isn't expensive, fetching it is. So if we fetch it all, then call a series of these smaller methods, we'd have a much easier time testing them, modifying them later, and extending the number that there are.
why dis comment?
from . import api, views
Ugh, that does complicate things more. I was trying to think of a case
where having the extra fields would be disadvantageous outside of
translator test cases, but I can't come up with a good use case, so I guess
cleaning them up in TranslatorTester may be sufficient.

On Thu, Mar 1, 2012 at 19:38, Simon Kornblith <
reply@reply.github.com
> wrote:

> > @@ -295,6 +307,14 @@ function addHighwireMetadata(doc, newItem) {
> >       newItem.accessDate = 'CURRENT_TIMESTAMP';
> >
> >       newItem.libraryCatalog = doc.location.host;
> > +
> > +     //clean up item
> > +     for(var field in newItem) {
> > +             if(!fieldIsValidForType(field, _itemType)) {
> > +                     delete newItem[field];
>
> But you're right that it's a good idea to cut out invalid fields when we
> make the test cases and map base fields to the item-specific field. I'll
> implement that in the test harness.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/zotero/translators/pull/138/files#r509979
>
@aurimasv - it looks as if there is no metadata on the test page anymore - while you update this, could you also include a new test and delete the old one - (this will do, e.g. for google/highwire http://tools.chass.ncsu.edu/open_journal/index.php/acontracorriente/article/view/174 )
Is this the best spot for this code?  Why let the camera get out of sync?  Perhaps the camera should know when the morph occurs and the `SceneState` changes.  If we let logic like this get scattered throughout the engine, the vast majority of people will have no hopes of understanding and modifying the rendering engine.

Part of this is because the Columbus view architecture is, erm, temporary, but I will be working on that soon, maybe even today.
Another option - but more verbose - is to prefix functions with `create` - `createFromQuaternion`, `createFieldOfView`, etc.  This way all the creation functions are together.  However, this can hurt the common case if that is a `from` function.

No change for now is OK with me.
And match this too.
Replace `billboard` with `polyline` throughout.
Script?
rubyのバージョンを1.9.1p378にあげたらrake成功しました。
ありがとうございます。
This code is moved to the registerForKeyPress() method which is called from TiViewProxy.handleGetView(). This design uses the similar logic as registerForTouch().
activeTab is not the same as selectedTab. activeTab can be 1 of 3 things: selectedTab, firstTab, or null. If activeTab is null, we know that selectedTab is null AND tabs array is empty (meaning the tabGroup contains no tab). You should look at handleGetActiveTab() for clarification.
Because the doc format was developed before there was a Mobile Web. And on Android and iOS, all titanium types extend a class called Proxy, which has certain characteristics that set it apart from normal JS objects.

So, FWIW, I find the MobileWeb equivalent not very intuitive ;-).
Needs to be `result === undefined`. You may also want to read this fascinating bit of literature about how `undefined` is not a JS keyword, and can in fact be redefined:

http://stackoverflow.com/questions/27509/detecting-an-undefined-object-property-in-javascript
Should include `section` data.
Why aren't these lines indenting properly? Spaces?
I think vec3 is a function from your library. If it's a class, it should be uppercase. Are you violating the standard guide because you want the API to be exactly the same as that used in other languages?
The idea here was to allow using the swift cookbook outside of a crowbar environment. A user could specify a different dsik_enum_expr than where crowbar stores disks (and a mechanism to filter on which disks to use)
This means, that the URL of an entry changes, depending on your timezone. I don't think that's right. If you publish a link to a post, then it might be not working for people in different timezones.
Ahh... reminds me. Passwords with `:` characters? Might be worth a test. Does Basic Auth urlencode? I don't see a decode.
Ziqi, I'm curious why you need to find the zero of a non-linear equation here. Not thinking too deep about it, it seems like this calc would be straight forward. You know the yaw angle of the rear frame, the roll angle and the steer angle...so then you only have to project the front wheel plane into the ground. Can you explain this too me?
i had written the function to get the list of packages and repos, but as this is not a part of the backend, i will remove it
There is no need for the underscore here, I think. We don't export BaseService, so nobody has access to it.
That's fine. Should the dialog be created lazily as well? Otherwise it would create the tab-less panel and a dialog instance on page load anyway.
So is this object representing one category?  If so, the class name should be singular.
isn't ``sym.owner`` risky? (I mean executing it outside of the PC)
Why is this needed? `immutable(T[])` is not a range, so I see no reason to try and support it in a range-based function, and if you use `immutable(T[])` with IFTI, then the template is instantiated with `immutable(T)[]`, so the range-based function actually takes the array as a mutable slice, meaning that a range-based function would never operate on a fully `immutable` array. So, I don't think that this change makes any sense.
Why this line? Surely dmd has to be smart enough to only call getcwd the first time base is used?
It's an issue with the implementation of `lazy` (for instance, `lazy` functions can't be inlined). It really should be fixed, but I have no idea if or when that will really happen. Andrei is a huge fan of `enforce` and generally insists on using it everywhere, but if performance is critical, it really isn't a good idea to use it  because of the performance issues with `lazy`. I've definitely seen `enforce` be the most expensive part of an algorithm when profiling code, and it's probably harming Phobos' performance a fair bit at this point given how much it's used (though it then theoretically becomes a big motivator for fixing `lazy`'s performance).

But yes, the `if` statement is a better alternative when performance is critical. Long term, I'd love to be able to use `enforce` in performance-critical code, but at this point, I don't think that it makes sense. And the whole point of this function is to improve performance.
(and `alias compose!(retro, reduce!commonPrefix, map!retro) commonSuffix;`)
Huge inattention from my side, will add.
Fixed all other comments.  For this one, the right way to do it is actually
to write
//foo.com
instead of forcing http or https.  That way you always get the appropriate
version... browsers warn about https loaded from http pages and vice versa.

On Tue, Aug 21, 2012 at 2:49 PM, John Messerly <notifications@github.com>wrote:

> In components_polyfill_boostrap.dart:
>
> > +  }
> > +  sb.add("""
> > +$sbMain
> > +  polyfill.initializeComponents();
> > +}
> > +""");
> > +
> > +  // Inject the output dart script on the page and the dart.js bootstrap script.
> > +  document.head.nodes
> > +   ..add(
> > +     new ScriptElement()
> > +       ..text = "$sb"
> > +       ..type = "application/dart")
> > +    ..add(
> > +      new ScriptElement()
> > +        ..src = "http://dart.googlecode.com/svn/branches/bleeding_edge/dart/client/dart.js"
>
> does it work if you use "https" url here?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dart-lang/dart-web-components/pull/16/files#r1430030>.
>
>
it'd be good to use logger here. Also I would log the stack trace here & below.
I believe war's can have the file in META-INF/beans.xml too.

So should just check both.
hummm... you do some mistakes here : node["twdc"]["cookbook"] :/
Can we avoid duplicating the data in memory and maybe write inplace?
That's was just unfinished work ;)
Does this need to be explicit with the parameter naming? The ordering is the same as the ctor definition.
i18nify '3 courses'
You stub sleep here so that you don't actually have to sleep.
nsIContentPolicy does not contain REJECT_METALINK. http://mxr.mozilla.org/mozilla-central/source/content/base/public/nsIContentPolicy.idl
forgot to delete it was a comment for me when I began localizing the changes to add
Ich würde einen Konstruktor schreiben und bei diesem den $order mitgeben...
Für den Order dann nur einen Getter schreiben (keinen Setter)

Ich denke, es darf nicht erlaubt sein, ein AccountsReceivable auf einen anderen Auftrag um zu biegen....
Впрочем, согласен, так лучше будет.
i didn't moved this, maybe is the diff on the file lenght?
What do you think about using LZO compression to store this? It would also have to be decompressed in SparseGetVersion.
MINOR: Refering to output_file param, not lf in comment.
I would use the word "determine" or "resolve" here. First, both are fairly common in the broader Spring codebase. Second, if the goal is to differentiate from "get", the word "obtain" is actually quite a close synonym.
But doListKeys() is called from 2 places getMessageCount

    this.doListKeys(MESSAGE_KEY_PREFIX + "*");

and iterator

    this.doListKeys(MESSAGE_GROUP_KEY_PREFIX + "*").iterator();

In any case, as noted above, we shouldn't be doing this when called from getMessageCount()

Seems to me that the stripping should be done in iterator() instead of here. Given this is not a public API, I don't see any reason to do it unconditionally here.
SortByFullName
The comment was for: `<log level="DEBUG"/>`  Should that be removed?
These files are auto generated by the Cordova/PhoneGap plugin.
I don't know why it says that. :P
This whole block belong in the API section, not the Usage section.
I'm doing a flot extension code above and a collection of helper functions (in the datastream object for now). I didn't decide yet how to embed this help functions. Should it be a class with constructor or just a collection ... They could in principle be packed as a jquery extension but some of them should not be run on any object, for example metricList, metricName ... We really have to think about this, because this is the API that we (and other users) will use for plotting.
the current version is  beautifulsoup4-4.0.0b10 
We shouldn't pin a beta version 
Or should we?
this test is useless here, since you requier `$data` to be `\stdClass` in function header
No, it is set by Vagrant itself. Is the best and safe way I know to be sure that Chef is running inside the Vagrant VM.
We can remove “currently in beta trial” too.
I think this makes sense.
it should - at least for ORM it works, just used it the other day. but yes, we should say "repository to use for documents extending this superclass"
Throw an SlmGoogleAnalytics\Exception\InvalidArgumentException with the message $domain_name is not  a string
``print >>`` is deprecated, I think? ``sys.stderr.write`` is probably what you want (though keep in mind, unlike print you need to add a ``\n`` at the end yourself)
wrong indentation for the content of the preview block
You should only need to specify the encoding in a file that actually contains UTF-8 characters. Why did you need to include it here?
we should probably call super() here and also use latest() instead of order_by + [0] (more readible)

also tests are needed
This is interesting. Is there any reason to have templated 'from' and 'extra_headers' fields?
If you want do define some specific headers then why not to do it in settings, or in database, or right in place of the message sending code?
For consistency, this is usually written as two lines in uh code. It's of course not a mistake the way you did it, but it would be great to always do it the same way.
s/Pysical/Physical
@sam-falvo To clarify what you mentioned about args[-1], I believe that referencing a non-existent index on an array will return undefined so the check will fail.
Ha, payload.

What does it do? And what does status do?
We need to find a better name for this.
Setting -> Sets
haven't ran coder, but needs space between dots
Good catch! That can't be right.

I've removed the "true". The rest was there in your version, so I left it. I'd agree that empty() would make sense, the only reason I can think of that it might not be that already is if someone was working around $version being 0 (zero), which empty returns true for? Even then though, I can't imagine why a new version would ever be set to "0".
i've opened #37 so we don't forget. you can assign it to me if you want ;)
Done. That was PhpStorm messing about
in our case, we just want 2 classes(PimCalendar and AccountFolderManger) have ability of mutex lock ( it equivalents that we define mutex_lock and mutex_unlock in each class separately) , and thread_sync never will have an instance, and no need to access the lock outside the subclass.
Should we do this check only if chunkedMode is true?
`s/Supported/are supported/`
I don't understand what this method want to do.